Skip to content

trigger CI - #308

Closed
raybellwaves wants to merge 5 commits into
mainfrom
raybellwaves-trigger-ci
Closed

trigger CI#308
raybellwaves wants to merge 5 commits into
mainfrom
raybellwaves-trigger-ci

Conversation

@raybellwaves

@raybellwaves raybellwaves commented May 7, 2021

Copy link
Copy Markdown
Member

this brings in

xarray==0.17.0 # latest is 0.18.0
xhistogram==0.1.2 # latest is 0.2.0
dask==2021.4.1
numpy==1.20.2

Pinned xhistogram to 0.1.2 (see xgcm/xhistogram#48)
Pinned xarray to 0.17.0 (xarray released yesterday on http://xarray.pydata.org/en/stable/whats-new.html#v0-18-0-6-may-2021)

@aaronspring how's climpred?

Pinning xarray fixed it. Will have to find a minimal example.

@raybellwaves raybellwaves mentioned this pull request May 7, 2021
10 tasks
Comment thread ci/minimum-tests.yml
# xhistogram 0.1.3 introduced an error that broke xskillscore
# see https://github.com/xgcm/xhistogram/issues/48
- xhistogram!=0.1.3
- xhistogram==0.1.2

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dougiesquire released xh=0.2.0

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah fails on xh==0.1.2 as well as xh latest. Guessing it's an xarray/dask/numpy update

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry guys, still waiting on a fix from dask for xhistogram

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah no worries mate. I created an issue for it here #311

@codecov

codecov Bot commented May 7, 2021

Copy link
Copy Markdown

Codecov Report

Merging #308 (d892ffc) into main (e2e3e7f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #308   +/-   ##
=======================================
  Coverage   94.50%   94.50%           
=======================================
  Files          23       23           
  Lines        2566     2566           
=======================================
  Hits         2425     2425           
  Misses        141      141           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 67b47b9...d892ffc. Read the comment docs.

@raybellwaves raybellwaves mentioned this pull request May 8, 2021
@raybellwaves
raybellwaves deleted the raybellwaves-trigger-ci branch May 9, 2021 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants