Skip to content

Speed up build time using micromamba#4121

Merged
IAlibay merged 22 commits intodevelopfrom
faster-ci
Apr 27, 2023
Merged

Speed up build time using micromamba#4121
IAlibay merged 22 commits intodevelopfrom
faster-ci

Conversation

@IAlibay
Copy link
Copy Markdown
Member

@IAlibay IAlibay commented Apr 9, 2023

Let's see how much time we can save (particularly on macos) by switching to micromamba.

In this PR:

  • Improved build speeds by ~ 50% (although in practice only saves ~ 30-50s per runner total).
  • Removed unecessary runners / moved them to Cron CI
  • Add old ubuntu + macos scan to cron CI.

PR Checklist

  • Tests?
  • Docs?
  • CHANGELOG updated?
  • Issue raised/referenced?

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

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2023

Linter Bot Results:

Hi @IAlibay! Thanks for making this PR. We linted your code and found the following:

There are currently no issues detected! 🎉

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 9, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (30baa14) 93.59% compared to head (6f2e50f) 93.59%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #4121   +/-   ##
========================================
  Coverage    93.59%   93.59%           
========================================
  Files          192      192           
  Lines        25134    25134           
  Branches      4056     4056           
========================================
  Hits         23524    23524           
  Misses        1092     1092           
  Partials       518      518           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@IAlibay IAlibay mentioned this pull request Apr 9, 2023
4 tasks
@IAlibay IAlibay marked this pull request as ready for review April 9, 2023 22:14
@IAlibay IAlibay changed the title [wip] Speed up build time using micromamba Speed up build time using micromamba Apr 9, 2023
@IAlibay IAlibay added this to the 2.5.0 milestone Apr 10, 2023
Copy link
Copy Markdown
Member

@hmacdope hmacdope left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One question but other than that looks good and everything green.

extra-specs: |
python=="3.11"
pip
channels: jaimergp/label/unsupported-cudatoolkit-shim, conda-forge, bioconda
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Question what needs jaimergp/label/unsupported-cudatoolkit-shim?

Copy link
Copy Markdown
Member Author

@IAlibay IAlibay Apr 26, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OpenMM otherwise we're pulling down cuda everywhere. - it's quite widely used everywhere (in CI) nowadays

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool beans

@IAlibay IAlibay merged commit 3d04453 into develop Apr 27, 2023
@IAlibay IAlibay deleted the faster-ci branch April 27, 2023 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants