-
Notifications
You must be signed in to change notification settings - Fork 9
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
C3s mapping #108
C3s mapping #108
Conversation
Esgf test data fix
Test time invariant datasets
@Zeitsperre Would like to merge this today, just checking its ok by you |
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.
Looks good. Let me make one or two changes and merge away!
try: | ||
url = "/".join((github_url, "raw", branch, md5name.as_posix())) | ||
LOGGER.info("Fetching remote file md5: %s" % fullname.as_posix()) | ||
LOGGER.info("Fetching remote file md5: %s" % md5name.as_posix()) |
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.
Nice catch, will need to fix this in other projects as well!
@ellesmith88 My changes are in and tests are passing locally. You can merge this as soon as the build clears or tomorrow morning! Thanks! EDIT: One final change to do! |
Pull Request Checklist:
bumpversion minor
has been called on this branchAUTHORS.md
What kind of change does this PR introduce?:
Small changes to allow datasets without a time dimension to pass through
Gets rid of submodule for mini-esgf-data
use
open_xr_dataset
fromroocs_utils.xarray_utils.xarray_utils
inclisops.ops.subset
but asexpand_wildcards
always returns a list, this will not affectops.subset
Does this PR introduce a breaking change?:
Using master branch of
roocs-utils
temporarily - changes in this PR will only work with this branchpyyaml>=5.3.*,<5.4
in requirements so documents buildOther information: