Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: improve test workflow performance #333

Merged
merged 2 commits into from
Jul 24, 2024
Merged

ci: improve test workflow performance #333

merged 2 commits into from
Jul 24, 2024

Conversation

afuetterer
Copy link
Contributor

This PR does two things. Please have a look at it and tell me what you think.

  1. it speeds up the package and dependency installation by using uv instead of pip
  2. it runs the pytest suite in parallel

Copy link

codecov bot commented Jul 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.00%. Comparing base (4735725) to head (77727c7).
Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #333   +/-   ##
=======================================
  Coverage   86.00%   86.00%           
=======================================
  Files          17       17           
  Lines        4209     4209           
  Branches      989      989           
=======================================
  Hits         3620     3620           
  Misses        430      430           
  Partials      159      159           

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

Copy link
Member

@lkstrp lkstrp left a comment

Choose a reason for hiding this comment

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

uv ❤️

Looks great, thanks a lot as always @afuetterer
Would you mind adding the changes to PyPSA/powerplantmatching#182?

All the repos I'll update after a powerplantmatching release.

@lkstrp lkstrp merged commit af63cdd into PyPSA:master Jul 24, 2024
16 checks passed
@afuetterer afuetterer deleted the uv branch July 24, 2024 14:22
@afuetterer
Copy link
Contributor Author

Sometimes there are a few hickups during the ci run, especially with the windows runners? It should take more than 2-3 minutes per test run.

@FabianHofmann
Copy link
Collaborator

looks great, we had a small printout of tested solvers in the log before, https://github.com/PyPSA/linopy/actions/runs/10074914889/job/27852237849#step:10:38

any chance to revive that again or have a logger info statement?

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.

3 participants