Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

santisoler
Copy link
Member

Summary

Experiment with caching the conda environment we need to create in Azure to
run simpeg tests. This way we could reduce the runtime of CI by reusing the
created environment.

PR Checklist

  • If this is a work in progress PR, set as a Draft PR
  • Linted my code according to the style guides.
  • Added tests to verify changes to the code.
  • Added necessary documentation to any new functions/classes following the
    expect style.
  • Marked as ready for review (if this is was a draft PR), and converted
    to a Pull Request
  • Tagged @simpeg/simpeg-developers when ready for review.

Reference issue

What does this implement/fix?

Additional information

Copy link

codecov bot commented Jun 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.20%. Comparing base (31fb6fe) to head (6a1202a).
Report is 3 commits behind head on main.

Current head 6a1202a differs from pull request most recent head 8a9a37d

Please upload reports for the commit 8a9a37d to get more accurate results.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1495   +/-   ##
=======================================
  Coverage   86.20%   86.20%           
=======================================
  Files         396      396           
  Lines       50341    50341           
  Branches     5311     5311           
=======================================
  Hits        43396    43396           
  Misses       5492     5492           
  Partials     1453     1453           

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

@jcapriot
Copy link
Member

You’ll need to explicitly pin the version of everything in the CI environment to make this correct.

@santisoler
Copy link
Member Author

You’ll need to explicitly pin the version of everything in the CI environment to make this correct.

Right. I think what I want to achieve is to create a single environment within each run. In that case I need to create an artifact instead of caching the environment.

@santisoler santisoler marked this pull request as draft June 24, 2024 21:19
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.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.