Feature 534 wind rose

[bikegeek]

Pull Request Testing

• Describe testing already performed for these changes:
  

using the test data and config file for the wind rose, modify values for the radialaxis_step and radialaxis_range. Also omit the radialaxis_range and verify a plot is still generated using Plotly's ranges.

• Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:
  
  generate plots using relevant data

Config file used in testing:
wind_rose_custom.yaml.txt

to generate this plot:

• Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? [NA]

• Do these changes include sufficient testing updates? [NA]

• Will this PR result in changes to the test suite? [No]
  
  If yes, describe the new output and/or changes to the existing output:
  

• Do these changes introduce new SonarQube findings? [Yes]
  
  If yes, please describe:
  SonarQube gate check will fail due to coverage value that falls below threshold value.

• Please complete this pull request review by [Fill in date].
  

Pull Request Checklist

See the METplus Workflow for details.

• Add any new Python packages to the METplus Components Python Requirements table.
• Review the source issue metadata (required labels, projects, and milestone).
• Complete the PR definition above.
• Ensure the PR title matches the feature or bugfix branch name.
• Define the PR metadata, as permissions allow.
  Select: Reviewer(s) and Development issue
  Select: Milestone as the version that will include these changes
  Select: Coordinated METplus-X.Y Support project for bugfix releases or METplotpy-X.Y.Z Development project for official releases
• After submitting the PR, select the ⚙️ icon in the Development section of the right hand sidebar. Search for the issue that this PR will close and select it, if it is not already selected.
• After the PR is approved, merge your changes. If permissions do not allow this, request that the reviewer do the merge.
• Close the linked issue and delete your feature or bugfix branch from GitHub.

[CPKalb]

I'm unclear exactly how the settings in the yaml file are supposed to work and so wanted to verify here. If I set the following:
radialaxis_range: [0, 15]
radialaxis_step: 5

then I get the wind rose below with 0, 5, 10, and 15% labeled as expected.

However, if I instead set the following:
radialaxis_range: [0, 15]
radialaxis_step: 3

then I get the wind rose below with 2, 5, 8, 11, and 14% labeled.

In this second case, I was expecting 0, 3, 6, 9, 12, 15%. What is the expected behavior for this second image?

[bikegeek]

I explicitly set dtick0 to the first value in the radialaxis_range and now the plot looks like this when I set the step size to 3:

#  Uncomment to specify the range of the radial axes to draw
radialaxis_range: [0, 15]

# Step size in between ticks of the axis
radialaxis_step: 3

[bikegeek]

I looked through the Plotly documentation for polar and polar bar plots and cannot get the last radial line drawn!

[CPKalb]

I explicitly set dtick0 to the first value in the radialaxis_range and now the plot looks like this when I set the step size to 3:

#  Uncomment to specify the range of the radial axes to draw
radialaxis_range: [0, 15]

# Step size in between ticks of the axis
radialaxis_step: 3

Is this a change I need to make somewhere (set dtick0 to 0)?

[CPKalb]

Tested on my data and works as expected, now having the ability to specify the range of the radial axes.

[bikegeek]

The radialaxis_step is the dtick0 value

…

On Mon, Aug 11, 2025 at 12:48 PM Christina Kalb ***@***.***> wrote: *CPKalb* left a comment (dtcenter/METplotpy#535) <#535 (comment)> I explicitly set dtick0 to the first value in the radialaxis_range and now the plot looks like this when I set the step size to 3: # Uncomment to specify the range of the radial axes to draw radialaxis_range: [0, 15] # Step size in between ticks of the axis radialaxis_step: 3 [image: wind_rose_custom] <https://private-user-images.githubusercontent.com/3753118/476392853-7e48d8bb-8297-4fe0-b8aa-4b068c268ae3.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NTQ5MzgzMDksIm5iZiI6MTc1NDkzODAwOSwicGF0aCI6Ii8zNzUzMTE4LzQ3NjM5Mjg1My03ZTQ4ZDhiYi04Mjk3LTRmZTAtYjhhYS00YjA2OGMyNjhhZTMucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI1MDgxMSUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNTA4MTFUMTg0NjQ5WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9OTNlM2JhN2Y5NGVkMDhkZTA2OTY1OGNkYThhNWVmZjY1MDEyODg2MTEwODRmODI2YmMxNGM4ZmQxZTM5Mzc5MCZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QifQ.ZFEwa0WkAzqsVJ7m3RXgpWhXj_-BriAkmICDdBB4rOw> Is this a change I need to make somewhere (set dtick0 to 0)? — Reply to this email directly, view it on GitHub <#535 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AA4UJHW2GDDUGAJJGDX5YAT3NDQOPAVCNFSM6AAAAACDC3637SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTCNZWGM2TINJRG4> . You are receiving this because you authored the thread.Message ID: ***@***.***>