Skip to content

GsoC Fetch Main Pull Request#5396

Open
jauy123 wants to merge 1 commit into
MDAnalysis:developfrom
jauy123:fetcher_gsoc
Open

GsoC Fetch Main Pull Request#5396
jauy123 wants to merge 1 commit into
MDAnalysis:developfrom
jauy123:fetcher_gsoc

Conversation

@jauy123
Copy link
Copy Markdown
Contributor

@jauy123 jauy123 commented Jun 5, 2026

Per our discussion (@talagayev @BradyAJohnston @yuxuanzhuang ) on Tuesday (6/2/2026), this is the main PR for the whole Fetch module. Each major step will have a separate pull request from this main request.

Changes made in this Pull Request:

LLM / AI generated code disclosure

LLMs or other AI-powered tools (beyond simple IDE use cases) were used in this contribution: yes / no

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!)
  • LLM/AI disclosure was updated.

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.

@read-the-docs-community
Copy link
Copy Markdown

read-the-docs-community Bot commented Jun 5, 2026

Documentation build overview

📚 MDAnalysis | 🛠️ Build #33011847 | 📁 Comparing e3bc2ae against latest (941d679)

  🔍 Preview build  

2 files changed
± index.html
± documentation_pages/analysis/wbridge_analysis.html

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.85%. Comparing base (941d679) to head (e3bc2ae).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #5396   +/-   ##
========================================
  Coverage    93.85%   93.85%           
========================================
  Files          182      182           
  Lines        22509    22509           
  Branches      3202     3202           
========================================
  Hits         21125    21125           
  Misses         922      922           
  Partials       462      462           

☔ View full report in Codecov by Harness.
📢 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.

@jauy123
Copy link
Copy Markdown
Contributor Author

jauy123 commented Jun 5, 2026

I think the branched off pull requests for this has to live in my MDAnalysis fork repo since this PR is trying to merge from my fork? I don't know enough about git to say this. Can someone clarify for this to me?

If so, the first nested PR is here (jauy123#1)

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.

1 participant