Skip to content

Conversation

orbeckst
Copy link
Member

@orbeckst orbeckst commented Jul 3, 2025

Fixes #5075

Changes made in this Pull Request:

  • document attributes n_frames, start, stop, step

PR Checklist

  • Issue raised/referenced?
  • Tests updated/added?
  • Documentation updated/added?
  • package/CHANGELOG file updated?
  • Is your name in package/AUTHORS? (If it is not, add it!)

Developers Certificate of Origin

I certify that I can submit this code contribution as described in the Developer Certificate of Origin, under the MDAnalysis LICENSE.


📚 Documentation preview 📚: https://mdanalysis--5076.org.readthedocs.build/en/5076/

Copy link

codecov bot commented Jul 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.63%. Comparing base (b113fd5) to head (ce14e6e).
Report is 2 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #5076   +/-   ##
========================================
  Coverage    93.62%   93.63%           
========================================
  Files          177      177           
  Lines        22001    22033   +32     
  Branches      3114     3115    +1     
========================================
+ Hits         20599    20631   +32     
- Misses         947      948    +1     
+ Partials       455      454    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

- fix #5075
- document attributes n_frames, start, stop, step
@orbeckst orbeckst force-pushed the fix-analysisbasedocs-5075 branch from 52d23cc to 98d7e6e Compare July 8, 2025 20:25
orbeckst and others added 2 commits July 14, 2025 10:04
Co-authored-by: Rocco Meli <r.meli@bluemail.ch>
@orbeckst
Copy link
Member Author

Thanks for looking at the PR & suggestions, which I merged.

Do you mind me being PR shepherd?

@RMeli RMeli merged commit d412c9a into develop Jul 14, 2025
23 checks passed
@RMeli RMeli deleted the fix-analysisbasedocs-5075 branch July 14, 2025 19:25
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.

document all generated attributes of AnalysisBase
2 participants