NatLabRockies / NatLabRockies/SAM
Names in Module Library are Missing Periods
Open
Beginner friendly
Nobody has claimed this yet.
bug
- Dominant language
- C++
- Stars
- 486
- Forks
- 219
- Avg merge
- 10h 10m
- Merged PRs (30d)
- 2
Description
Module names in the module library are missing periods. This was intended to remove unneccessary information like the period in "Ltd." but had the unintended consequence of removing periods from numbers like SolarEdge SE333KUS, which should be SE33.3KUS to indicate a 33.3 kW inverter.
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 module-library name formatting responsible for removing periods. Reproduce or inspect the SolarEdge SE333KUS case and verify that numeric names retain the decimal point while the intended unnecessary punctuation handling remains correct. Done when module names display the expected periods.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 65/100