NatLabRockies / NatLabRockies/SAM
Comma in Wind Turbine Name
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 486
- Forks
- 219
- Avg merge
- 10h 10m
- Merged PRs (30d)
- 2
Description
Describe the bug
The name for the Fortis Passaat turbine in the wind turbine library uses a comma instead of period to indicate the decimal value "1.4 kW" preventing the power curve data from loading correctly.
To Reproduce
- Create a wind case.
- On the Wind Turbine page, filter "passaat" and select the Fortis Passaat turbine. Note the empty power curve graph.
Workaround is to edit /libraries/Wind Turbines.csv file and change Line 20 "Fortis Passaat 3.12m 1,4kW " to "Fortis Passaat 3.12m 1.4kW".
Operating System and Version
Tested in SAM 2026.7.3, probably also in older versions.
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
Open /libraries/Wind Turbines.csv and inspect line 20, the Fortis Passaat 3.12m 1,4kW entry. Reproduce the empty power curve by selecting Fortis Passaat in a wind case, then correct the decimal separator and verify that the power curve loads.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- data
- Issue type
- Bug
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Quiet
- Clarity
- Clearly specified
- Newbie friendliness
- 88/100