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

BLD: restrct pandas version - fix test error #259

Merged
merged 1 commit into from Jul 5, 2021

Conversation

hongyeehh
Copy link
Member

The github action error originated from a pandas update and triggered a tqdm function calling error.

This shall be solved in the next tqdm release next release v4.61.2 #1180, until then we could limit our pandas version to pandas <= 1.2.5

Also, the numpy update to v1.21 introduces new warnings in our test function DEP: Deprecate error clearing for special method in array-coercion #19001, which I think relates to shapely methods. We need to wait for shapely to suppress the warnings

@codecov
Copy link

codecov bot commented Jul 5, 2021

Codecov Report

Merging #259 (fc8e034) into master (215c6fc) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #259   +/-   ##
=======================================
  Coverage   95.00%   95.00%           
=======================================
  Files          38       38           
  Lines        1583     1583           
  Branches      260      260           
=======================================
  Hits         1504     1504           
  Misses         41       41           
  Partials       38       38           

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 215c6fc...fc8e034. Read the comment docs.

@hongyeehh hongyeehh merged commit 0588812 into mie-lab:master Jul 5, 2021
@hongyeehh hongyeehh deleted the fix-test-error branch July 5, 2021 14:49
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.

None yet

1 participant