Skip to content
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

Deprecate Configuration.library_path #122

Merged
merged 2 commits into from
Oct 23, 2023
Merged

Conversation

RHammond2
Copy link
Contributor

This PR deprecates the duplicated requirement for a library_path in Configuration and Simulation with the most significant change coming from Simulation.from_config, which now requires 2 inputs library_path and config.

@RHammond2 RHammond2 added the enhancement New feature or request label Oct 23, 2023
@codecov
Copy link

codecov bot commented Oct 23, 2023

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (e4591f4) 53.73% compared to head (086dc0f) 53.76%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #122      +/-   ##
===========================================
+ Coverage    53.73%   53.76%   +0.03%     
===========================================
  Files           22       22              
  Lines         3726     3722       -4     
  Branches       767      765       -2     
===========================================
- Hits          2002     2001       -1     
+ Misses        1602     1599       -3     
  Partials       122      122              
Files Coverage Δ
wombat/core/simulation_api.py 51.32% <33.33%> (+0.90%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@RHammond2 RHammond2 merged commit 20fcb13 into develop Oct 23, 2023
5 of 6 checks passed
@RHammond2 RHammond2 deleted the enhancement/library-config branch October 23, 2023 19:58
RHammond2 added a commit that referenced this pull request Oct 27, 2023
* Update Test PyPI action settings (#108)

* Bug Fix: Process recreation and equipment cost breakdowns (#109)

* update equipment labor breakdown to include equipment and duration

* add missing process recreation for in situ replacements

* update docstring

* update changelog

* fix bad reference (#110)

* Bug Fix: Ensure system downtime starts appropriately (#111)

* only reset subassemblies that are worked on directly

* allow invalid to happen before servicing and properly control

* update the operating level logging in in_situ_repairs

* ensure repairs actually go into the next shift if incomplete from weather delays

* ensure travel can't occur when it goes past one shift and into another

* fix repair manager checks for updated logic

* ignore third party's deprecation warnings

* fix the unscheduled timing for better logic control

* ensure a delayed start to the repair still counts as the initial processing

* slight timing update for handling of 24 hour shifts

* ensure 24 hour shifts don't travel back when new window needed

* remove print

* put system invalidation back in place

* update changelog

* ensure requests can't collide

* move wait period to the request getter

* fix combined yield and return

* fix tests

* add invalidation to timing collision avoidance

* make system invalidation more flexible

* move random wait to environment

* remove timeout from service equipment

* ensure port repairs always trigger before the next at site repair

* ensure ttp are not dispatched multiple times for the same turbine

* fix hlv dispatch timing

* add list of systems waiting for ttp to reduce chances of overlapping dispatches

* update changelog

* Deprecate old library (#114)

* remove old library warnings and fully deprecate the usage

* update the changelog

* update the how to guide for the library changes

* Deprecate PySAM Usage (#117)

* remove pysam

* remove pysam references from the docs

* Bug fix: Task completion rate clarity (#118)

* add potential fix for low task completion due to replacements

* log which request type is canceled in 'action'

* update metric for change in canceled phrasing

* fix cable getter to allow for multiple cable naming conventions

* update changelog

* Bug Fix: Keep logged events within the timing bounds (#120)

* ensure events attempting to be logged after the end of simulation are cancelled

* update changelog

* add NAWEA 2023 documentation

* add working COREWIND D4.2 data

* Feature: Add basic plotting routines (#121)

* add basic plotting functionality

* allow for time or enery avail and fix memory carryover bug

* update changelog

* add windfarm operations calculation at the front end and fix windfarm productino bug

* update changelog

* update changelog

* add example and update workshop guidance

* fix end of file lines

* update data for labor costs and update workshop listings

* Deprecate `Configuration.library_path` (#122)

* depreacte configuration library requirement

* update documentation

* update workshop guide

* add ORBIT port costs

* update the presentation and notebook

* OpEx Breakdown Function  (#119)

* Update post_processor.py

OpEx breakdown function

* eliminate extra year

* delete nan years line

* make existing opex() method make the breakdown

* Opex breakdown in Metrics.opex()

* add CHANGELOG

---------

Co-authored-by: dmulash <[email protected]>

* remove presentation from gh

* Enhancement: Switch to `pyproject.toml` for all configurations (#123)

* update wombat urls

* remove old config and update python version

* rerun pre-commit hooks

* fully upgrade to pyproject.toml

* update pyprpoject to install actual package and data

* Update documentation ahead of v0.9 release (#124)

* udpate documentation examples

* update outdated explanation

* update outputs

* fix bad hyperlink

* udpate changelog

* split up the api documentation and utilities

* add plotting methods

* fix typos and enable utilities docs'

* fix redundant docs

* remove extra params'

* udpate changelog

* fix example bugs

* update example results

* minor fixes

* update examples listing

* ignore examples in auto running

* update user guide and opex with breakdown

* update the example notebooks

* fix bad vessel naming convention

* fix issues with config compatibility

* rerun validation notebooks and update timing

* remove results from repository to save space

* update licensing per NREL-Apache guidance

* update the license notice on the docs

* update copyright date in docs

* update examples notebook pages

* rerun examples to get output to show

* remove links to the presentation until approved

---------

Co-authored-by: Daniel Mulas Hernando <[email protected]>
Co-authored-by: dmulash <[email protected]>
github-actions bot pushed a commit that referenced this pull request Oct 27, 2023
* Update Test PyPI action settings (#108)

* Bug Fix: Process recreation and equipment cost breakdowns (#109)

* update equipment labor breakdown to include equipment and duration

* add missing process recreation for in situ replacements

* update docstring

* update changelog

* fix bad reference (#110)

* Bug Fix: Ensure system downtime starts appropriately (#111)

* only reset subassemblies that are worked on directly

* allow invalid to happen before servicing and properly control

* update the operating level logging in in_situ_repairs

* ensure repairs actually go into the next shift if incomplete from weather delays

* ensure travel can't occur when it goes past one shift and into another

* fix repair manager checks for updated logic

* ignore third party's deprecation warnings

* fix the unscheduled timing for better logic control

* ensure a delayed start to the repair still counts as the initial processing

* slight timing update for handling of 24 hour shifts

* ensure 24 hour shifts don't travel back when new window needed

* remove print

* put system invalidation back in place

* update changelog

* ensure requests can't collide

* move wait period to the request getter

* fix combined yield and return

* fix tests

* add invalidation to timing collision avoidance

* make system invalidation more flexible

* move random wait to environment

* remove timeout from service equipment

* ensure port repairs always trigger before the next at site repair

* ensure ttp are not dispatched multiple times for the same turbine

* fix hlv dispatch timing

* add list of systems waiting for ttp to reduce chances of overlapping dispatches

* update changelog

* Deprecate old library (#114)

* remove old library warnings and fully deprecate the usage

* update the changelog

* update the how to guide for the library changes

* Deprecate PySAM Usage (#117)

* remove pysam

* remove pysam references from the docs

* Bug fix: Task completion rate clarity (#118)

* add potential fix for low task completion due to replacements

* log which request type is canceled in 'action'

* update metric for change in canceled phrasing

* fix cable getter to allow for multiple cable naming conventions

* update changelog

* Bug Fix: Keep logged events within the timing bounds (#120)

* ensure events attempting to be logged after the end of simulation are cancelled

* update changelog

* add NAWEA 2023 documentation

* add working COREWIND D4.2 data

* Feature: Add basic plotting routines (#121)

* add basic plotting functionality

* allow for time or enery avail and fix memory carryover bug

* update changelog

* add windfarm operations calculation at the front end and fix windfarm productino bug

* update changelog

* update changelog

* add example and update workshop guidance

* fix end of file lines

* update data for labor costs and update workshop listings

* Deprecate `Configuration.library_path` (#122)

* depreacte configuration library requirement

* update documentation

* update workshop guide

* add ORBIT port costs

* update the presentation and notebook

* OpEx Breakdown Function  (#119)

* Update post_processor.py

OpEx breakdown function

* eliminate extra year

* delete nan years line

* make existing opex() method make the breakdown

* Opex breakdown in Metrics.opex()

* add CHANGELOG

---------

Co-authored-by: dmulash <[email protected]>

* remove presentation from gh

* Enhancement: Switch to `pyproject.toml` for all configurations (#123)

* update wombat urls

* remove old config and update python version

* rerun pre-commit hooks

* fully upgrade to pyproject.toml

* update pyprpoject to install actual package and data

* Update documentation ahead of v0.9 release (#124)

* udpate documentation examples

* update outdated explanation

* update outputs

* fix bad hyperlink

* udpate changelog

* split up the api documentation and utilities

* add plotting methods

* fix typos and enable utilities docs'

* fix redundant docs

* remove extra params'

* udpate changelog

* fix example bugs

* update example results

* minor fixes

* update examples listing

* ignore examples in auto running

* update user guide and opex with breakdown

* update the example notebooks

* fix bad vessel naming convention

* fix issues with config compatibility

* rerun validation notebooks and update timing

* remove results from repository to save space

* update licensing per NREL-Apache guidance

* update the license notice on the docs

* update copyright date in docs

* update examples notebook pages

* rerun examples to get output to show

* remove links to the presentation until approved

---------

Co-authored-by: Daniel Mulas Hernando <[email protected]>
Co-authored-by: dmulash <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant