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

TST: mask warning in test_cf_from_numpy_dtypes #783

Merged
merged 1 commit into from
Mar 1, 2021

Conversation

snowman2
Copy link
Member

@snowman2 snowman2 commented Mar 1, 2021

Continuation of #775

@codecov
Copy link

codecov bot commented Mar 1, 2021

Codecov Report

Merging #783 (af2b7b5) into master (6b860de) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #783   +/-   ##
=======================================
  Coverage   94.69%   94.69%           
=======================================
  Files          21       21           
  Lines        1528     1528           
=======================================
  Hits         1447     1447           
  Misses         81       81           

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 6b860de...af2b7b5. Read the comment docs.

@snowman2 snowman2 merged commit 68f79af into pyproj4:master Mar 1, 2021
@snowman2 snowman2 deleted the tst branch March 1, 2021 02:00
@@ -51,19 +51,20 @@ def test_cf_from_numpy_dtypes():
"latitude_of_projection_origin": numpy.int(45),
Copy link
Contributor

@jorisvandenbossche jorisvandenbossche Mar 1, 2021

Choose a reason for hiding this comment

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

Instead of masking the warning, I think the "proper" fix is to change the above to numpy.int64(45) (because the above usage of np.int() is deprecated)

Copy link
Member Author

Choose a reason for hiding this comment

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

The warning masked is the one about losing CRS information raised by pyproj when exporting to PROJ string. That is good to know about the numpy deprecation. If you have a moment to raise an issue or open a quick PR with the fix, that would be helpful. Thanks 👍

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, yes, stupid me, of course the warning you were fixing wasn't coming from here, since the pytest.warns is only below ;)

(but indeed, something to fix anyway, and I see you already did a PR!)

sthagen added a commit to sthagen/pyproj4-pyproj that referenced this pull request Mar 1, 2021
TST: mask warning in test_cf_from_numpy_dtypes (pyproj4#783)
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.

2 participants