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

Reinforcement costs now computed and reported per-MW #174

Merged
merged 1 commit into from
Feb 6, 2023

Conversation

ppinchuk
Copy link
Collaborator

@ppinchuk ppinchuk commented Feb 3, 2023

After taking a much closer look at the first round of reinforcement cost results, Trieu et al. decided that smaller wind/solar farms should not have to pay the same reinforcement costs as larger wind/solar farms. Instead, they wanted the cost to be a fraction proportional to the farm size, so the reinforcement costs are now computed and reported as $/MW instead of a flat dollar amount, which can then scale with the built-out plant size.

@ppinchuk ppinchuk mentioned this pull request Feb 3, 2023
@codecov-commenter
Copy link

codecov-commenter commented Feb 3, 2023

Codecov Report

Merging #174 (c8da5cd) into main (70ba350) will decrease coverage by 0.01%.
The diff coverage is 92.30%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #174      +/-   ##
==========================================
- Coverage   81.11%   81.11%   -0.01%     
==========================================
  Files         118      118              
  Lines       13728    13730       +2     
==========================================
+ Hits        11136    11137       +1     
- Misses       2592     2593       +1     
Flag Coverage Δ
unittests 81.11% <92.30%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...eVX/least_cost_xmission/least_cost_xmission_cli.py 51.69% <0.00%> (ø)
reVX/least_cost_xmission/least_cost_paths.py 97.71% <100.00%> (-0.04%) ⬇️
reVX/least_cost_xmission/trans_cap_costs.py 86.15% <100.00%> (+0.06%) ⬆️
tests/test_xmission_least_cost_paths.py 95.71% <100.00%> (ø)
reVX/handlers/database.py 0.00% <0.00%> (ø)
reVX/rpm/rpm_clusters.py 80.34% <0.00%> (+0.08%) ⬆️
reVX/plexos/plexos_plants.py 58.95% <0.00%> (+0.09%) ⬆️

@ppinchuk ppinchuk merged commit f147ec9 into main Feb 6, 2023
@ppinchuk ppinchuk deleted the pp/scaled_reinforcement_costs branch February 6, 2023 16:25
github-actions bot pushed a commit that referenced this pull request Feb 6, 2023
Reinforcement costs now computed and reported per-MW
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.

None yet

3 participants