NREL / reV

Renewable Energy Potential (reV) Model
https://nrel.github.io/reV/
BSD 3-Clause "New" or "Revised" License
107 stars 24 forks source link

Bespoke site data to sys inputs #392

Closed ppinchuk closed 1 year ago

ppinchuk commented 1 year ago

Added ability to input site-specific SAM technology inputs via the project points csv input in Bespoke. Also added support for capital cost, fixed operating cost, and variable operating cost multiplier inputs, also via project points.

Also simplified some of the code (no logic changes) in the PlaceTurbines class.

No new tests were necessary - changes were small enough to fit within the existing test infrastructure.

codecov-commenter commented 1 year ago

Codecov Report

Base: 80.60% // Head: 80.91% // Increases project coverage by +0.30% :tada:

Coverage data is based on head (8e89142) compared to base (4ce3970). Patch coverage: 98.90% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #392 +/- ## ========================================== + Coverage 80.60% 80.91% +0.30% ========================================== Files 143 143 Lines 19900 19927 +27 ========================================== + Hits 16040 16123 +83 + Misses 3860 3804 -56 ``` | Flag | Coverage Δ | | |---|---|---| | unittests | `80.91% <98.90%> (+0.30%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#carryforward-flags-in-the-pull-request-comment) to find out more. | [Impacted Files](https://codecov.io/gh/NREL/reV/pull/392?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL) | Coverage Δ | | |---|---|---| | [reV/bespoke/place\_turbines.py](https://codecov.io/gh/NREL/reV/pull/392/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-cmVWL2Jlc3Bva2UvcGxhY2VfdHVyYmluZXMucHk=) | `96.34% <98.41%> (+4.64%)` | :arrow_up: | | [reV/bespoke/bespoke.py](https://codecov.io/gh/NREL/reV/pull/392/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-cmVWL2Jlc3Bva2UvYmVzcG9rZS5weQ==) | `89.41% <100.00%> (+0.01%)` | :arrow_up: | | [tests/test\_bespoke.py](https://codecov.io/gh/NREL/reV/pull/392/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-dGVzdHMvdGVzdF9iZXNwb2tlLnB5) | `100.00% <100.00%> (+7.15%)` | :arrow_up: | | [reV/losses/scheduled.py](https://codecov.io/gh/NREL/reV/pull/392/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-cmVWL2xvc3Nlcy9zY2hlZHVsZWQucHk=) | `100.00% <0.00%> (ø)` | | | [reV/losses/power\_curve.py](https://codecov.io/gh/NREL/reV/pull/392/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-cmVWL2xvc3Nlcy9wb3dlcl9jdXJ2ZS5weQ==) | `98.86% <0.00%> (+<0.01%)` | :arrow_up: | | [reV/config/project\_points.py](https://codecov.io/gh/NREL/reV/pull/392/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-cmVWL2NvbmZpZy9wcm9qZWN0X3BvaW50cy5weQ==) | `85.41% <0.00%> (+0.04%)` | :arrow_up: | | [reV/handlers/collection.py](https://codecov.io/gh/NREL/reV/pull/392/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-cmVWL2hhbmRsZXJzL2NvbGxlY3Rpb24ucHk=) | `80.11% <0.00%> (+0.05%)` | :arrow_up: | | [tests/test\_hybrids.py](https://codecov.io/gh/NREL/reV/pull/392/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-dGVzdHMvdGVzdF9oeWJyaWRzLnB5) | `96.43% <0.00%> (+0.23%)` | :arrow_up: | | [reV/config/base\_config.py](https://codecov.io/gh/NREL/reV/pull/392/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-cmVWL2NvbmZpZy9iYXNlX2NvbmZpZy5weQ==) | `84.92% <0.00%> (+0.24%)` | :arrow_up: | | [tests/test\_gen\_pv.py](https://codecov.io/gh/NREL/reV/pull/392/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL#diff-dGVzdHMvdGVzdF9nZW5fcHYucHk=) | `94.92% <0.00%> (+0.36%)` | :arrow_up: | | ... and [6 more](https://codecov.io/gh/NREL/reV/pull/392/diff?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL) | | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NREL)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.