PV Design UI Improvements#1352
Conversation
* Remove duplicate `inv_snl_inverter_name` inverter library variable, was used in inverter_not_available_msg() function to check for missing library items, but this triggers on_change that prevents new error-checking of editable inverter voltage inputs on System Design page. Missing library items should only happen for users who edit the inverter library, which is a special case for advanced users. * Add call to System Design page reset_inverter_voltages() function to update inputs on System Design page when inverter voltage ratings on Inverter page change. * Remove duplicate inverter name variable from CEC Information section. Inverter name is above plot, consistent with Module page. * Change variable label for module and inverter library from "module/inverter name" to "module/inverter library selection" for clarity.
…inv_cec_name` to correct default.
sjanzou
left a comment
There was a problem hiding this comment.
Nice sizing message and popup when Vdc_max is low
Copy ratings from inverter page button is useful to reset
I am confused on the MPPT functionality - setting MPPT to 3 on inverter page and then clicking the Set MPPT inputs on the System Design page as follows:
Gives a simulation error:
project file attached
SAM_1352_Multiple_MPPTs.zip
What am I doing wrong?
|
@sjanzou I tried to improve the multiple MPPT messages to help guide users through confusing workflows -- this could use a UI redesign, but hopefully these messages work for now. Let me know if you find any other confusing workflows that are not handled clearly enough. |
sjanzou
left a comment
There was a problem hiding this comment.
Okay, 1 inverter for MPPT - I get that now!
Everything else looks good and the voltage settings are a great addition to test different scenarios
Pull Request Template
Description
For configurations with Detailed PV model, make inverter voltage ratings editable on System Design page so users can correct voltage ratings from CEC databsae that don't match manufacturer datasheets.
Improve PV sizing messages with more detailed suggestions:
Make multiple MPPT option more transparent:
To test:
Fixes #1020
Partially addresses #209
Type of change
Please delete options that are not relevant.
Checklist:
If you have added a new compute module in a SSC pull request related to this one, be sure to check the Process Requirements.