Skip to content

Better enforce "don't export to grid" constraints#550

Merged
brtietz merged 1 commit into
patchfrom
ssc_513_enforce_no_grid_export
Mar 18, 2021
Merged

Better enforce "don't export to grid" constraints#550
brtietz merged 1 commit into
patchfrom
ssc_513_enforce_no_grid_export

Conversation

@brtietz

@brtietz brtietz commented Mar 12, 2021

Copy link
Copy Markdown
Collaborator

Two changes: First - pull adjustments out of else-if chain to ensure they are applied consistently. This also helps increase charging targets in price signals dispatch. Second, adjust the custom dispatch targets if they are too high, reducing iteration. Adjust tests as needed

  • Includes a new test to trigger this specific bug
  • Changes to two other custom dispatch tests were expected
  • Price signals dispatch now has better access to charging power, due to a related constraint in the if-else chain
  • The BattWatts changes were unexpected, but justified when I checked the tests in the debugger

…d. First - pull adjustments out of else-if chain to ensure they are applied consistently. This also helps increase charging targets in price signals dispatch. Second, adjust the custom dispatch targets if they are too high, reducing iteration. Adjust tests as needed
@brtietz

brtietz commented Mar 12, 2021

Copy link
Copy Markdown
Collaborator Author

Companion SAM PR: NatLabRockies/SAM#561

@dguittet

Copy link
Copy Markdown
Collaborator

@brtietz There was a failing test-- did you check that?

@brtietz

brtietz commented Mar 18, 2021

Copy link
Copy Markdown
Collaborator Author

No, I didn't spot it until after I merged it into patch. Fix went directly to patch: fdb3272

Price signals has some minor differences running between Windows and Linux, I haven't had time to debug it yet.

@brtietz brtietz deleted the ssc_513_enforce_no_grid_export branch May 25, 2021 21:59
sjanzou added a commit that referenced this pull request Nov 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants