-
Notifications
You must be signed in to change notification settings - Fork 659
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
Dask timeseries prototype #4714
base: develop
Are you sure you want to change the base?
Conversation
Hello @ljwoods2! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2024-10-02 22:17:36 UTC |
Linter Bot Results:Hi @ljwoods2! Thanks for making this PR. We linted your code and found the following: Some issues were found with the formatting of your code.
Please have a look at the Please note: The |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #4714 +/- ##
===========================================
- Coverage 93.54% 92.92% -0.63%
===========================================
Files 173 174 +1
Lines 21434 21519 +85
Branches 3981 3991 +10
===========================================
- Hits 20051 19997 -54
- Misses 929 1059 +130
- Partials 454 463 +9 ☔ View full report in Codecov by Sentry. |
Fixes #4713
Changes made in this Pull Request:
H5MDReader
to expose adasktimeseries
methodPR Checklist
Developers certificate of origin
📚 Documentation preview 📚: https://mdanalysis--4714.org.readthedocs.build/en/4714/