Add "Allen method" to Subhourly Clipping Correction Option#1861
Conversation
|
@cpaulgilman is it ok to add to this branch to add a switch between Allen and Walker methods? It would likely be a dropdown with Allen method and Walker method as the text in the dropdown menu. |
Yes -- consider radio buttons in case dropdown suggests preference for one option over the other. Also, is there a way to differentiate the methods with a descriptive term instead of the developer's name? |
@cpaulgilman I would appreciate notes on the wording here. We can provide references to Allen and Walker in the documentation. Walker method: https://www.nrel.gov/docs/fy20osti/76859.pdf |
|
@mjprilliman Thanks for adding the Walker method. I've redesigned the UI and revised Help: |
Looks great Paul! My only note would be that only the Allen method I would describe as empirical, so I would probably remove that adjective there. |
Good catch. I'll remove "empirical" from the general description. |





Pull Request Template
Description
Note Help revisions for the "Losses" topic will be committed separately before the Fall 2024 release. Please review and comment on following screenshot.
Fixes #1850 (issue)
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.