Skip to content
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

Modified non_parcel_bunkers_motion_experimental to use non pressure weighted mean wind #243

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

collin-volk
Copy link

Per issue #209, it appears as though non_parcel_bunkers_motion_experimental is the function most similar to the process described in Bunkers et al. 2000 in that it uses the shear vector from the 0-0.5km mean wind to the 5.5-6km mean wind. The issue is that it was using the 0-6km pressure weighted mean wind in the calculation rather than the non pressure weighted mean wind which is recommended in Bunkers et al. 2000 and reaffirmed in Bunkers et al. 2014. This change modifies the function to use the non-pressure weighted mean wind. If the intention was to use pressure weighted mean wind, it should probably be the 0-8km mean wind as this was found to minimize error in Bunkers et al. 2014. This change does not solve the broader problem brought up by #209, namely that the bunkers motion calculations in SHARPpy use the 0-6km shear vector rather than the shear vector from the 0-0.5km mean wind to the 5.5-6km mean wind, but I was reluctant to make any larger changes as I'm not sure if this is actually a bug or if there was a valid reason as to why the 0-6km shear is being used.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant