Batt btm to grid and manual clip#754
Conversation
… still need to do manual discharge to grid
1. Reorganize Discharge inputs for manual dispatch.
2. Also for manual dispatch, fix “to grid” remains checked while “discharge” is unchecked.
3. Disable "battery can discharge to grid" when it is not used in dispatch decision.
* Discharge to grid should be shown for custom (input battery power targets) and manual.
* It should be hidden for peak shaving, input grid power targets, and price signal forecast.
4. Fix versions.lk to address #577.
…/NREL/SAM into batt_btm_to_grid_and_manual_clip
* Also updates some other defaults in battery configurations that should all be checked once things are merged into Develop and more stable.
…o_grid_and_manual_clip
mjprilliman
left a comment
There was a problem hiding this comment.
A few questions and UI quirks, but otherwise looks good. I'm trusting ssc to test result changes for different charging and discharging conditions
| @@ -1,5 +1,5 @@ | |||
| 1 | |||
| 1058 | |||
| 1073 | |||
There was a problem hiding this comment.
Defaults note: PV Battery Commercial has manual schedule with 4 weekday periods and only 2 periods enabled. Manual dispatch is not the default dispatch mode for PV Battery Commercial
There was a problem hiding this comment.
Peak shaving as the default dispatch is intentional.
Great point about only having two periods of discharging, I'll get that updated such that it always discharges during the high cost period.
|
|
||
| 0 | ||
| 2642 | ||
| 2567 |
There was a problem hiding this comment.
The labels are a bit unclear is charging from clipped power supposed to be allowed for a custom time series?
There was a problem hiding this comment.
This is a bug, clipped power should be disabled for custom time series. I'll get this fixed in the next push.
| else | ||
| { show('batt_dispatch_auto_can_fuelcellcharge', false); } | ||
| enable('batt_dispatch_auto_can_charge',option != 3); | ||
| enable('batt_dispatch_auto_can_gridcharge', option !=3 ); |
There was a problem hiding this comment.
Fuel Cell Single Owner might need some enable checks
There was a problem hiding this comment.
Page load order is not correct for Fuel Cell Single Owner
…o_grid_and_manual_clip
…dispatch_choice_ui methodology. make the commercial battery manual dispatch default schedule match up with the high cost periods in the default utility rate
|
@cpaulgilman Are we intending a battery paired with the fuel cell to be able to discharge to grid in this release? If so, did we want to add those to this PR or a subsequent PR? I also made some changes to the enable/disable behavior to better reflect the powerflow calculations (which are used by everything, including fuel cell) so this might be worth another look from you. |
Pairs with SSC PR NatLabRockies/ssc#653
Adds UI options to enable disable clip charging and discharging to grid for behind the meter dispatch, and FOM manual dispatch.