-
Notifications
You must be signed in to change notification settings - Fork 132
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Validation of results using tests #52
Comments
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. |
Maybe it is to much for the restructuring meeting but we should adopt a roadmap there. Shall we change the milestone to that meeting? |
perhaps it would be better suited for the oemof developer meeting in may. what do you think? |
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. |
I would shift that topic to the re-factoring meeting. |
@ckaldemeyer Do you think this is issue is covered by PR #160 and #154? Otherwise you should specify the remaining topics. |
@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? |
Sorry, just hit the wrong button 😄 |
Another idea: would a method in the EnergySystem class make sense? If not, I would tend to do it in solph! |
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. |
👍 |
Maybe we can shedule this to the release after the refactoring-release as I won't be able to commit myself before September.. |
I guess this can be closed, or not? |
Or at least be split in two kinds of tests |
I gues this can be closed, or not? |
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. |
@oemof/oemof-developer-group Please keep it in mind that there is a wiki like this. |
Within the development-phase of renpass-gis, results have to be validated. Since this is a general procedure for most models, the idea was to implement this using tests. If suitable, these tests can then be implemented into oemof_base afterwards.
Here are some quick ideas to start with (any further suggestions are welcome):
@oemof/oemof-main : To you have any suggestions where to implement this? Using nose directly in the test folder?
@simonhilpert : Did I forget something?
@uvchik : I remember you telling me that you already had testing procedures at the RLI. Feel free to contribute!
The text was updated successfully, but these errors were encountered: