-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add analyse module #10
Conversation
uvchik
commented
Jul 2, 2020
•
edited
Loading
edited
- Analyse scenario
- merit order
- Analyse results
- merit order
- Overall emissions
- specific emissions
- hourly emissions
- overall costs
- produced electricity
Hello @uvchik! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2020-09-07 09:42:43 UTC |
This pull request introduces 3 alerts when merging 67673f9 into b10df51 - view on LGTM.com new alerts:
|
…x into festures/add-analyse-module
The doctest are skipped because they do need data which is not available.
…x into festures/add-analyse-module
…x into festures/add-analyse-module
This pull request introduces 1 alert when merging 8ceb334 into b10df51 - view on LGTM.com new alerts:
|
…x into festures/add-analyse-module
This pull request introduces 1 alert when merging c534f9e into b10df51 - view on LGTM.com new alerts:
|
…x into festures/add-analyse-module
This pull request introduces 1 alert when merging 051aa6e into b10df51 - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging a441ee1 into b10df51 - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging dbe569e into b10df51 - view on LGTM.com new alerts:
|
…x into festures/add-analyse-module