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

Update 'real' argument for xarray #88

Merged
merged 2 commits into from
Feb 15, 2020
Merged

Update 'real' argument for xarray #88

merged 2 commits into from
Feb 15, 2020

Conversation

roxyboy
Copy link
Member

@roxyboy roxyboy commented Nov 19, 2019

I think this should fix issues #78 and #83 .

@codecov-io
Copy link

codecov-io commented Nov 19, 2019

Codecov Report

Merging #88 into master will increase coverage by 0.21%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #88      +/-   ##
==========================================
+ Coverage   96.87%   97.08%   +0.21%     
==========================================
  Files           2        2              
  Lines         737      756      +19     
  Branches      126      131       +5     
==========================================
+ Hits          714      734      +20     
+ Misses         13       12       -1     
  Partials       10       10
Impacted Files Coverage Δ
xrft/xrft.py 94.84% <100%> (+0.09%) ⬆️
xrft/tests/test_xrft.py 99.01% <0%> (+0.28%) ⬆️

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 43835e1...52de754. Read the comment docs.

@navidcy
Copy link
Contributor

navidcy commented Nov 26, 2019

@roxyboy, seems to be working fine now.

@roxyboy
Copy link
Member Author

roxyboy commented Nov 26, 2019

Can I merge this @rabernat ?

@rabernat
Copy link
Collaborator

rabernat commented Feb 8, 2020

Sorry for letting this hang for so long without any action.

Every time we fix a bug, we need to add a regression test. So I will merge this as soon as you add a test for the issues raised in #78 and #83.

@@ -17,7 +17,7 @@ notifications:

python:
- 2.7
- 3.6
- 3.7
Copy link
Collaborator

Choose a reason for hiding this comment

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

Keep 3.6 please but also add 3.7.

@rabernat rabernat merged commit 09be983 into xgcm:master Feb 15, 2020
@navidcy
Copy link
Contributor

navidcy commented Mar 21, 2020

@roxyboy could you please publish a new patch release of the package?

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.

4 participants