Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Polar plot method #183

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

Open
Tom-Kingstone wants to merge 7 commits into develop
base: develop
Choose a base branch
Loading
from Python_Toolkit-#179-PolarCoordinatePlot

Conversation

@Tom-Kingstone
Copy link
Contributor

@Tom-Kingstone Tom-Kingstone commented Jul 1, 2025

NOTE: Depends on

Issues addressed by this PR

Closes #179

moved polar plot method from LadybugTools_Toolkit to here. Also added ability to view the summed totals of values for use with plotting anything other than wind roses (for instance a radiation rose would be more useful to show total radiation from a direction, rather than the probability it happens), implemented through the density bool.

The implementation is necessarily different to how it works in LBT (due to not being linked to a dataset through the Wind class), so data is input as a dataframe, and the method reads the values and directions columns specified.

Removed other_data as an input, as now that was a workaround for the Wind class to create roses from data that was not in the wind object itself (basically moving the polar plot method to Python_Toolkit made it redundant). See the linked LBT PR for more info.

Renamed other_bins to value_bins (as it is easier to tell what it actually means this way)

Test files

run the unit tests in LBT Toolkit
When unit tests for this method are made, run those as well.

Do a spot check with some openmeteo data in jupyter to check that it all works.

Changelog

  • Moved polar plot method to this toolkit from LadybugTools_Toolkit

Additional comments

@Tom-Kingstone Tom-Kingstone added status:WIP PR in progress and still in draft, not ready for formal review type:feature New capability or enhancement labels Jul 1, 2025
@Tom-Kingstone Tom-Kingstone removed the status:WIP PR in progress and still in draft, not ready for formal review label Jul 7, 2025
Copy link
Contributor Author

@BHoMBot check installer -force

Copy link

bhombot-ci bot commented Jul 10, 2025

@Tom-Kingstone to confirm, the following actions are now queued:

  • check installer

Copy link

bhombot-ci bot commented Jul 10, 2025

The check installer has already been run previously and recorded as a successful check. This check has not been run again at this time.

Tom-Kingstone reacted with thumbs down emoji

Copy link
Contributor Author

@BHoMBot check installer -force

Copy link

bhombot-ci bot commented Jul 10, 2025

@Tom-Kingstone to confirm, the following actions are now queued:

  • check installer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jamesramsden-bh jamesramsden-bh Awaiting requested review from jamesramsden-bh

@CKBoulter CKBoulter Awaiting requested review from CKBoulter

@tg359 tg359 Awaiting requested review from tg359

At least 1 approving review is required to merge this pull request.

Labels

type:feature New capability or enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Move PlotWindrose from LBT_TK to Python Toolkit

2 participants

AltStyle によって変換されたページ (->オリジナル) /