aics-int / napari-allencell-annotator

Other
0 stars 1 forks source link

Remove BioReader #75

Open memeramita opened 1 month ago

memeramita commented 1 month ago

Currently bioio requires us to specify readers for different types of images when creating the BioImage object.

According to @BrianWhitneyAI in the future we will not have to specify readers for different types of images. We should remove our reader selection logic and let bioio determine the best reader to use if possible.

BrianWhitneyAI commented 3 weeks ago

I think this is resolved by bioio 3d16713 . If I remember there were three issues going on here. 1) that the wrong reader was being selected. That should be resolved. The second was an error being raised for the dump_plugins method. This also has been resolved in https://github.com/bioio-devs/bioio/pull/59 . lastly, there was an issue with "ome.tif" files not being recognized as ome.tiff. "ome.tif" was not one of our supported suffixes so this is expected. Ill have to ask around if we want to support 'ome.tif' (more likely why we wouldnt.