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

Make quality control plots a standard part of mesh generation for E3SM #96

Open
xylar opened this issue Apr 30, 2021 · 11 comments
Open
Assignees
Labels
enhancement New feature or request ocean

Comments

@xylar
Copy link
Collaborator

xylar commented Apr 30, 2021

As discussed here, it would be really helpful to have quality control plots (like here) generated automatically as part of creating meshes and initial conditions for E3SM.

This would presumably involve porting Matlab code to python as part of a new test case for the global ocean.

@xylar xylar added enhancement New feature or request ocean python package DEPRECATED: PRs and Issues involving the python package (master branch) labels Apr 30, 2021
@xylar
Copy link
Collaborator Author

xylar commented Apr 30, 2021

@proteanplanet, as a first step toward assessing how much work this would be, would you be willing to point me to your Matlab scripts so I can see how much code we're talking about and how much beyond my rusty Matlab skills this might be?

@xylar
Copy link
Collaborator Author

xylar commented Apr 30, 2021

cc @vanroekel

@dengwirda
Copy link
Contributor

Hi all, there are also a bunch of mesh quality routines available direct in jigsaw-python...

@xylar
Copy link
Collaborator Author

xylar commented Apr 30, 2021

Thanks @dengwirda. Those would also be helpful to include. I think what is most critical in this context are some things about quality of coast lines related to specific features that @proteanplanet has picked out. So what I have in mind here are more quality control from a geopolitical and physical oceanography standpoint. I'm guessing what jigsaw-python provides are metrics that are more related to numerical method? If so, that would still be a useful addition. If you have specific suggestions or examples you'd like to point us to, please do!

@dengwirda
Copy link
Contributor

dengwirda commented Apr 30, 2021

@xylar, yes totally agree --- @proteanplanet's coastline/channel quality control is very important on its own (and makes very nice plots!). I think it could be of interest to add additional metrics though, to ensure geometric quality is consistent with what TRSK is happy to deal with. These are some standard things that I run in the jigsaw examples, with a little more detail here.

@xylar
Copy link
Collaborator Author

xylar commented Apr 30, 2021

@dengwirda, got it. So these wouldn't be images saved out or fields added to the mesh necessarily but rather output to the log file when the mesh is being created, right?

@dengwirda
Copy link
Contributor

@xylar, I think there could be a few possibilities. Printing to the log-file makes sense, but another option could be to use the metrics to identify additional cells/regions (that fall below some quality thresholds, say) to be passed to @proteanplanet's workflow for additional visual inspection?

@xylar
Copy link
Collaborator Author

xylar commented Apr 30, 2021

Okay, so for that purpose, we would want to add these metrics to the mesh and pass them through at least to the culling stage.

@dengwirda
Copy link
Contributor

@xylar, that would definitely work, but I'm not sure what the easiest path might be at this stage --- another option is that the MPAS mesh data-structure may (I think) have enough information in it to just compute these metrics on the fly within/before @proteanplanet's workflow.
Does it make sense for the three of us to discuss a little more first, and try to hash out a design?

@xylar
Copy link
Collaborator Author

xylar commented Apr 30, 2021

The rest of this work is likely to be substantial. @proteanplanet's plotting scripts are, to the best of my knowledge, in Matlab. So porting them to Python will be a not insignificant task. I'm sure there will be time to discuss adding these metrics to that workflow. It likely won't happen in the immediate term, though.

@xylar
Copy link
Collaborator Author

xylar commented Aug 4, 2021

@proteanplanet, could you point us to your Matlab code for making your quality control plots if it is publicly available?

@xylar xylar removed the python package DEPRECATED: PRs and Issues involving the python package (master branch) label Apr 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ocean
Projects
None yet
Development

No branches or pull requests

3 participants