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

Add MpasMaskCreator back (temporarily) #533

Closed
milenaveneziani opened this issue Oct 27, 2023 · 3 comments · Fixed by #535
Closed

Add MpasMaskCreator back (temporarily) #533

milenaveneziani opened this issue Oct 27, 2023 · 3 comments · Fixed by #535

Comments

@milenaveneziani
Copy link
Contributor

The old MpasMaskCreator C++ tool was superseded by the new python tool in #514. For transect masks, the former does something that no other tool does at the moment though, and that is the ordering of the transect elements (say, edges) starting from the transect start point and ending with the transect end point. That is important for plotting routines that rely on transect edge masks to plot vertical sections (x vs z plots).
Until we have a plotting routine for cross-transect velocities that does not rely on the edge mask, it would be good to add the old C++ tool back.

@xylar
Copy link
Collaborator

xylar commented Oct 27, 2023

Thanks @milenaveneziani!

@milenaveneziani
Copy link
Contributor Author

Thanks @xylar. That was fast!

@xylar
Copy link
Collaborator

xylar commented Oct 28, 2023

Sure, no problem!

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

Successfully merging a pull request may close this issue.

2 participants