NatLabRockies / NatLabRockies/SAM
Many macros don't work for hybrid simulations
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 486
- Forks
- 219
- Avg merge
- 10h 10m
- Merged PRs (30d)
- 2
Description
Multiple macros need to be updated for hybrid simulations, or removed from those configs, because the variable names used in the macros do not match the hybrid naming structure (prepended with the technology name.
This is relevant in most of the macros on the list, and should require only minor updating but the config type does need to be checked.
Current workaround is to run the macro of interest in a case set up for a single generation technology, then port any relevant inputs to the hybrid case.
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
No files, tests, or specific macros are named. Locate the macro list and hybrid configuration definitions, compare the variable names used by each macro with the hybrid naming structure, and verify that each relevant macro works or is removed from incompatible configurations.
Written by the indexing model from the issue text.
Assessment
- Domain
- tooling
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100