Closed K-Meech closed 7 months ago
Attention: Patch coverage is 96.77419%
with 1 lines
in your changes are missing coverage. Please review.
Project coverage is 84.35%. Comparing base (
13d0444
) to head (0e4e956
). Report is 24 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
brainglobe_utils/image_io/utils.py | 92.30% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
What is this PR
Why is this PR needed?
brainreg
currently has aLoadFileException
class that should be refactored into brainglobe-utils, and merged with the existingImageIOLoadException
What does this PR do?
This PR moves the functionality of
LoadFileException
intoImageIOLoadException
, making sure the relevant load functions catch errors like: loading a single tiff from a directory, loading 2D tiffs etc.References
For https://github.com/brainglobe/brainreg/issues/180 Corresponding PR on
brainreg
https://github.com/brainglobe/brainreg/pull/186How has this PR been tested?
Tests were updated, and all pass locally.
Is this a breaking change?
No
Does this PR require an update to the documentation?
No
Checklist: