Implement pip dependency caching for Azure Pipelines#5267
Implement pip dependency caching for Azure Pipelines#5267unnatikdm wants to merge 7 commits intoMDAnalysis:developfrom
Conversation
There was a problem hiding this comment.
Hello there first time contributor! Welcome to the MDAnalysis community! We ask that all contributors abide by our Code of Conduct and that first time contributors introduce themselves on GitHub Discussions so we can get to know you. You can learn more about participating here. Please also add yourself to package/AUTHORS as part of this PR.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #5267 +/- ##
===========================================
- Coverage 93.83% 93.83% -0.01%
===========================================
Files 182 182
Lines 22501 22501
Branches 3195 3195
===========================================
- Hits 21114 21113 -1
- Misses 924 925 +1
Partials 463 463 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
855f366 to
ccae476
Compare
|
There is a PR template which includes a check list and required certification around origin of code (authoring and AI usage): https://github.com/MDAnalysis/mdanalysis/blob/develop/.github/PULL_REQUEST_TEMPLATE.md Please include it filled out in the PR description. Ensure you read the AI policy (https://github.com/MDAnalysis/mdanalysis/blob/develop/AI_POLICY.md) and answer truthfully. Currently a number of files have been added to git that are not required as they are results of compiling, please remove them. |
|
Hi @BradyAJohnston, thanks for the feedback! I've updated the PR description to include the full template with AI disclosure (truthfully stated that AI assistance was used for debugging and formatting). I also removed the accidentally added compiled/binary files in commit Regarding the Codecov coverage drop (-7.69%): It appears unrelated to my changes because only CI configuration and documentation files were modified. Codecov itself notes "No files covered by tests were changed" and that the commits have different numbers of coverage report uploads, which can cause artificial drops. I believe this can be safely ignored. Please let me know if any further changes are needed. Thanks again for your review! |
|
/azp run |
|
Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command. |
IAlibay
left a comment
There was a problem hiding this comment.
I'm going to put a block on this. It's not immediately clear to me that this actually brings about any significant improvements in CI install time.
That would have to be proven, and also a means of clearing the cache should be readily available before we consider moving ahead with this.
2749de8 to
7beb40f
Compare
|
@IAlibay I’ve addressed the feedback: Added CACHE_VERSION variable to allow manual cache clearing (incremented to force a fresh cache). This is included in both the primary key and the restore keys. |
Fixes #5255
Changes made in this Pull Request:
Implement pip dependency caching for Azure Pipelines to reduce CI runtime:
$(HOME)/.cache/pip, Windows:$(LocalAppData)\pip\Cache)LLM / AI generated code disclosure
LLMs or other AI-powered tools (beyond simple IDE use cases) were used in this contribution: yes
Disclosure details: I used AI assistance (ChatGPT) to help debug the Windows path quoting issue and to format the PR description. The core implementation logic and understanding of the problem were my own, with AI helping to:
All code was reviewed and understood before committing.
PR Checklist
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.