Closed adamltyson closed 5 months ago
I assume the tests are failing on macOS as those runners have recently been updated to ARM. #191 should fix this.
Good catch, thanks. I've set these to -99 which seems about right.
I think merge. This shouldn't (famous last words) affect only one OS.
Will hold off releasing until #191 though.
Description
This PR makes sure the outputs of the napari plugin and CLI are equal. Previously, as highlighted in #192 by @AdrianAriasAbreu, there were differences when running the same data in the napari plugin and CLI. +
Why is this PR needed? The plugin and CLI should be two ways of achieving the same thing.
What does this PR do?
brainglobe_utils.image_io.load.load_any()
(as used by the CLI).References
Closes #192 (I think)
How has this PR been tested?
The same data has been registered with the plugin and CLI and results confirmed to be the same. Existing tests pass.
Is this a breaking change?
No
Does this PR require an update to the documentation?
No, but I noticed some errors in the docs, updated in https://github.com/brainglobe/brainglobe.github.io/pull/170
Checklist: