search

brainglobe / brainglobe-workflows

Workflows that utilise BrainGlobe tools to perform data analysis and visualisation.
BSD 3-Clause "New" or "Revised" License
9 stars 2 forks source link

Avoid caching `pip` dependencies #124

Closed alessandrofelder closed 1 month ago

alessandrofelder commented 1 month ago

Description

What is this PR

Why is this PR needed?

If we run the benchmarks using the current configurations on a local machine, we risk pip using its cache. This stops the benchmarks from being useful smoke tests for newer versions of dependencies breaking our code, which is a key purpose of this repo.

What does this PR do? Stops pip from using cached dependencies when installing the package into the asv conda env.

References

\

How has this PR been tested?

Only understanding how pip works better. Suggestions for how to best test this welcome :thinking:

Is this a breaking change?

No.

Does this PR require an update to the documentation?

yes, but we should tackle that as a whole once we are happy with the workflows and our internal testing setup.

Checklist:

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.83%. Comparing base (9ad7474) to head (5b6ec05).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #124 +/- ## ======================================= Coverage 85.83% 85.83% ======================================= Files 17 17 Lines 847 847 ======================================= Hits 727 727 Misses 120 120 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.