Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Suppress UserWarning caused by MDAnalysis 0.18 #71

Merged
merged 2 commits into from
May 18, 2018
Merged

Conversation

mimischi
Copy link
Contributor

Changes made in this pull request:

  • Filter UserWarning caused by MDAnalysis 0.18

PR Checklist

  • [ ] Added changelog fragment in ./changelog/ (more information)?
  • [ ] Issue raised/referenced?

@mimischi
Copy link
Contributor Author

mimischi commented May 18, 2018

Actually now that h5py==2.8 is released, their FutureWarning is gone. I think we should leave in that old filter, because users may not update to the newest h5py version and we do not want to force a minimal version in the package.

https://github.com/h5py/h5py/releases/tag/2.8.0

Thoughts? @kain88-de

@codecov
Copy link

codecov bot commented May 18, 2018

Codecov Report

Merging #71 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #71      +/-   ##
==========================================
+ Coverage   90.35%   90.37%   +0.01%     
==========================================
  Files          12       12              
  Lines         508      509       +1     
==========================================
+ Hits          459      460       +1     
  Misses         49       49
Impacted Files Coverage Δ
mdbenchmark/__init__.py 100% <100%> (ø) ⬆️

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 b1e974d...995ae07. Read the comment docs.

@kain88-de kain88-de merged commit 466324d into master May 18, 2018
@mimischi mimischi deleted the userwarnings branch August 2, 2018 12:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants