Giter Site home page Giter Site logo

Comments (17)

uvchik avatar uvchik commented on June 15, 2024

I would differ between programming test and system validation tests. For me there are three kinds of test:

1.) The normal nose test should be very fast because you execute them quite often to check if the program itself works. The optimisation itself should not be part of these tests.

2.) Validation test to check if an optimisation still gets the same results after changing the code. These test should be done before a merge or at least before we will release. There should be a handful of representative models.

3.) Plausibility test to check the results of a new optimisation. They are optional but a good help if the models become more and more complicated.

I think your suggestions are of the third kind.

from oemof-solph.

uvchik avatar uvchik commented on June 15, 2024

Maybe it is to much for the restructuring meeting but we should adopt a roadmap there. Shall we change the milestone to that meeting?

from oemof-solph.

cswh avatar cswh commented on June 15, 2024

perhaps it would be better suited for the oemof developer meeting in may. what do you think?

from oemof-solph.

uvchik avatar uvchik commented on June 15, 2024

It is an important part of the quality management and people already broke other peoples app by pushing untested code. But it will be a matter of time, so let's see.

from oemof-solph.

uvchik avatar uvchik commented on June 15, 2024

I would shift that topic to the re-factoring meeting.

from oemof-solph.

uvchik avatar uvchik commented on June 15, 2024

@ckaldemeyer Do you think this is issue is covered by PR #160 and #154? Otherwise you should specify the remaining topics.

from oemof-solph.

uvchik avatar uvchik commented on June 15, 2024

@ckaldemeyer : Sorry I misunderstood the main point. I guess the main point is to implement plausibility tests. Do you think it should be a part of solph or a part of the outputlib?

from oemof-solph.

uvchik avatar uvchik commented on June 15, 2024

Sorry, just hit the wrong button 😄

from oemof-solph.

ckaldemeyer avatar ckaldemeyer commented on June 15, 2024

@ckaldemeyer : Sorry I misunderstood the main point. I guess the main point is to implement plausibility tests. Do you think it should be a part of solph or a part of the outputlib?

Another idea: would a method in the EnergySystem class make sense?

If not, I would tend to do it in solph!

from oemof-solph.

uvchik avatar uvchik commented on June 15, 2024

I think it depends on your approach but if it is part of the EnergySystem it should work for oemof in general. So think implementing it within solph will be easier.

from oemof-solph.

ckaldemeyer avatar ckaldemeyer commented on June 15, 2024

I think it depends on your approach but if it is part of the EnergySystem it should work for oemof in general. So think implementing it within solph will be easier.

👍

from oemof-solph.

ckaldemeyer avatar ckaldemeyer commented on June 15, 2024

Maybe we can shedule this to the release after the refactoring-release as I won't be able to commit myself before September..

from oemof-solph.

ckaldemeyer avatar ckaldemeyer commented on June 15, 2024

I guess this can be closed, or not?

from oemof-solph.

ckaldemeyer avatar ckaldemeyer commented on June 15, 2024

Or at least be split in two kinds of tests

from oemof-solph.

ckaldemeyer avatar ckaldemeyer commented on June 15, 2024

I gues this can be closed, or not?

from oemof-solph.

uvchik avatar uvchik commented on June 15, 2024

We do have code tests, example test.

We do not have plausibility tests and as I understand it this was your main concern according to your first comment.

The ideas of you first comment are quite interesting. Maybe we should add wiki to save these ideas but make it possible to close ancient issues.

https://github.com/oemof/oemof/wiki/ideas

from oemof-solph.

uvchik avatar uvchik commented on June 15, 2024

@oemof/oemof-developer-group Please keep it in mind that there is a wiki like this.

from oemof-solph.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.