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

Slim dependencies #119

Closed sfmig closed 2 weeks ago

sfmig commented 2 weeks ago

Description

What is this PR

Why is this PR needed? It seems like we have redundant dependencies in pyproject.toml but would like to double check.

What does this PR do? Remove dependencies that are included in brainglobe meta package.

References

Inspired by https://github.com/brainglobe/brainglobe-atlasapi/pull/310

How has this PR been tested?

Tests pass locally and on CI.

Is this a breaking change?

No.

Does this PR require an update to the documentation?

No.

Checklist:

codecov[bot] commented 2 weeks ago

Codecov Report

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

Project coverage is 85.83%. Comparing base (f9f62ab) to head (b09b234).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #119 +/- ## ======================================= 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.

adamltyson commented 2 weeks ago

Imio can also be removed (it's been merged into brainglobe-utils).