Skip to content
This repository has been archived by the owner on Dec 13, 2022. It is now read-only.

Add xesmf and xskillscore to environment #54

Closed
wants to merge 1 commit into from

Conversation

jthielen
Copy link
Contributor

Regridding (with xesmf) and verification metrics (with xskillscore) were planned as a part of the Advanced Model Output vs. Observations project idea, so this adds those two packages to the environment list. xesmf (with ESMF) is a fairly hefty dependency, so hopefully that doesn't cause too many issues. At least v0.5.0 of xesmf is required so that we get automatic lat/lon coordinate identification.

@dopplershift
Copy link
Member

Let's get #52 merged and try to rebase this on top--maybe that will allow the windows environment to resolve?

@jthielen
Copy link
Contributor Author

@dopplershift Doesn't look like that worked. But, with that environment update, the error is a bit easier to read, and it looks like (for whatever reason) we can't simulatenously have xesmf and xskillscore up-to-date on windows (xesmf=0.5.0 pins xarray to <= 0.16.0, whereas xskillscore requires at least 0.16.2).

I think the fix may have to petition xesmf for a 0.5.1 release working on 0.16.2 (since it was a 0.16.1 bug that was the reason for the pin).

@dopplershift
Copy link
Member

You can try to make the argument on the xesmf conda-forge feedstock.

@jthielen
Copy link
Contributor Author

jthielen commented Jan 8, 2021

Closed in favor of #94, since I don't think we'll be resolving the upstream issue in time.

@jthielen jthielen closed this Jan 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants