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

Merchant plant to battery forecast input sums inappropriately #825

Open
brtietz opened this issue May 27, 2022 · 1 comment
Open

Merchant plant to battery forecast input sums inappropriately #825

brtietz opened this issue May 27, 2022 · 1 comment

Comments

@brtietz
Copy link
Collaborator

brtietz commented May 27, 2022

Describe the bug
If you provide a price of $20/MWh to just the energy market in the merchant plant, the battery will receive a price for that timestep of $20/MWh. If you provide that same price to all 5 markets, those values will be summed, and the battery will assume it's receiving $100/MWh. While the percentage price differences remain the same between steps, the magnitude matters when comparing to the cost of cycle degradation.

To Reproduce
Steps to reproduce the behavior:

  1. Run the attached SAM file: aligned price series.zip
  2. Compare the "power price for battery dispatch" between the all energy market case and the all markets case.

Expected behavior
If we're only going to provide one price signal to battery dispatch, it should be scaled appropriately to the number of markets active. Discuss whether we can use a simple average or a weighted average, and if this needs to factor into the solution for NREL/SAM#181

@brtietz
Copy link
Collaborator Author

brtietz commented Sep 25, 2024

Easiest valid approach: weighted average. This has potential issues with cleared capacity as a MW value, where the battery might not decide to dispatch & fail to meet the cleared capacity. See if 181 can address this or if we need to do something else.

Battery price computation in a difference cmod: break this out of the battery code since it potentially contributes to financial model modularity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant