Closed adamltyson closed 4 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 90.43%. Comparing base (
6185777
) to head (38c21ac
).
: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?
pynrrd
seems to be causing a few issues with newer versions of Python (t doesn't seem to have been updated for 2 years). Mostly because it uses nptyping, which itself needs updating.However, I don't think we need nrrd IO functionality in brainglobe, because we only deal with tiffs and nii. We may use this functionality in the atlas generation code, but I think if we do, it should be a specific requirement for that script (and not for the whole of brainglobe as it is at the moment).
Keen to know if this is going to break anything, but I don't think it should.
What does this PR do? Removes all nrrd IO code. It also removes (what I think is) some outdated error/warning handling.
References
N/A
How has this PR been tested?
Removed nrrd tests, and ran other tests as normal.
Is this a breaking change?
I don't think so.
Does this PR require an update to the documentation?
Yes, the reference to nrrd IO needs to be removed from the website docs.
Checklist: