-
Notifications
You must be signed in to change notification settings - Fork 6
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
MPAS native output support + example #16
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
probably temporary, may not really be the best idea but otherwise get cannot pickle '_thread.lock' object in the joblib parallel setup normally don't use parallel for this anyway, since there are only 6 time steps and it is pretty fast in serial
since over 100 MB it hits the virus check page though
Also tried this longer command, which supposedly did it in the past ["wget", "--no-verbose", "--no-check-certificate", url, "-O", (HERE / fn).as_posix(), "-r", "-A", "'uc*'", "-e", "robots=off", "-nd"] but didn't work now
plotting the data and seeing how the contouring could work
zmoon
commented
Dec 8, 2023
they are the same geo area and such, might as well
Directory for sample MPAS native data:
Directory of mesh:
|
zmoon
commented
Feb 3, 2024
wget command for the large file that worked before doesn't anymore there are apparently workarounds that work in 2024 https://medium.com/geekculture/wget-large-files-from-google-drive-336ba2e1c991 but gdown seems the safer way to go
and hide the output of the first (import) cell so as not to see the ERROR 1: PROJ: proj_create_from_database ... failed message
note: had to use :func: or the functions stopped getting linked with the addition of a :doc: seems :ref: works fine without this modification, but then get annoying anchor links that aren't needed
saw this warning in RTD: tams/data.py:238: FutureWarning: The return type of `Dataset.dims` will be changed to return a set of dimension names in future, in order to be more consistent with `DataArray.dims`. To access a mapping from dimension names to lengths, please use `Dataset.sizes`.
and crossref the example also str agg wasn't working I think, but now it should but really should type it properly since single callable should be supported as well
knubez
reviewed
Feb 4, 2024
knubez
approved these changes
Feb 4, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really appreciate all of the descriptive comments and how accessible it is now.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
contours
andidentify
, assumed if only one spatial dimensionTriangulation
) + Matplotlibtricontour
Note
Should add
dask-core
andgdown
to the conda-forge recipe for this release.