NatLabRockies / NatLabRockies/H2Integrate
Update the PySAM battery model to take an input dictionary names `pysam_options` so that other battery model attributes can be set
Open
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 26
- Forks
- 44
- Avg merge
- 3d 22h
- Merged PRs (30d)
- 16
Description
(Optional) Update the PySAM battery model to take an input dictionary names pysam_options so that other battery model attributes can be set (similar to what's done in the PySAM solar model and the PySAM wind model to allow for more user configurability)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the PySAM solar and wind model implementations that already accept pysam_options, then compare their interfaces with the PySAM battery model. Add equivalent battery configurability and verify that the supported battery attributes can be set through the input dictionary.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100