Skip to content

Feature sensitivity analysis step and fix storage optimization#936

Merged
Bachibouzouk merged 10 commits intodevfrom
feature/sensitivity_analysis_step
Mar 30, 2022
Merged

Feature sensitivity analysis step and fix storage optimization#936
Bachibouzouk merged 10 commits intodevfrom
feature/sensitivity_analysis_step

Conversation

@Bachibouzouk
Copy link
Copy Markdown
Collaborator

@Bachibouzouk Bachibouzouk commented Mar 29, 2022

Changes proposed in this pull request:

  • Function server.run_sensitivity_analysis_step to perform one step of a sensitivity analysis
  • Function utils.nested_dict_crawler to return mapping of path within a nested dict to the keys at the lowest nested level
  • Enable capacity optimization for storage assets in the epa
  • Input timeseries is now not returned to epa in utils.data_parser.py
  • Add test class test_utils.TestAccessKPIs to test the nested dict utils functions
  • Make the utils function get_nested_valueand set_nested_value raise a Key error with a traceback indicating where in the nested dict this key was missing to help debugging

#923 has to be merged prior to merge this PR

The following steps were realized, as well (if applies):

  • Use in-line comments to explain your code
  • Write docstrings to your code (example docstring)
  • For new functionalities: Explain in readthedocs
  • Write test(s) for your new patch of code (pytests, assertion debug messages)
  • Update the CHANGELOG.md
  • Apply black (black . --exclude docs/)
  • Check if benchmark tests pass locally (EXECUTE_TESTS_ON=master pytest)

For more information on how to contribute check the CONTRIBUTING.md.

@Bachibouzouk Bachibouzouk mentioned this pull request Mar 29, 2022
9 tasks
@Bachibouzouk Bachibouzouk force-pushed the feature/sensitivity_analysis_step branch from 76bb12b to d3b9413 Compare March 30, 2022 09:48
@Bachibouzouk Bachibouzouk changed the title Feature/sensitivity analysis step Feature sensitivity analysis step and fix storage optimization Mar 30, 2022
@Bachibouzouk Bachibouzouk force-pushed the feature/sensitivity_analysis_step branch from 00edefc to 079f9eb Compare March 30, 2022 10:14
@Bachibouzouk Bachibouzouk requested a review from ciaradunks March 30, 2022 10:16
@Bachibouzouk Bachibouzouk merged commit 6ddd770 into dev Mar 30, 2022
@Bachibouzouk Bachibouzouk deleted the feature/sensitivity_analysis_step branch March 30, 2022 10:36
@Bachibouzouk Bachibouzouk mentioned this pull request Apr 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants