-
Notifications
You must be signed in to change notification settings - Fork 154
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use correct turbine names in plots #804
Comments
sorry looks like my screenshot is too fuzzy to read but @paulf81 can add the original image if needed! |
Hi @aclerc, thanks for bringing this up! I think the functionality you are after is actually available in FLASC. For example, take a look at this function, where you can pass However, this functionality is a bit hidden on FLASC. This ties in well with discussion #789, and is probably a point in favor of moving the functionality over to FLORIS. |
@aclerc can you share more information so that we can recreate this issue? A simplified analysis script including the functions you used for plotting would be helpful. Also, which version of FLORIS are you using? |
hi all, just to jump in, I think this is more enhancement than bug. Typically turbines in a farm have a name/identifier. In FLORIS and FLASC we use 0-index for each turbine, and so plotting functions typically label turbines according to that system. @misi9170 you're right this currently more a FLASC issue, but with the move to FLORIS, I think the main idea in this issue is that any plot that labels specific turbines, should have the option to use provided names in place of 0-index. A secondary point, is that if we label turbines 0-index we should probably not add the T-prefix, is that is usually how a named turbine would be specified |
Believe this issue is resolved by #805 |
I was just discussing this with @paulf81 and he encouraged me to add an issue :)
Please can any plots showing turbine names use the correct turbine names. For example in the farm shown below the turbines are actually named T02, T04, T05, etc. but T00, T01, T02, etc. is displayed. If the correct turbine names are not available using the name "T00" will probably never be right, so if you have to fall back to generated names I would at start with T01
The text was updated successfully, but these errors were encountered: