Sarah Rashidi
Sarah Rashidi
Hi @ericspod thanks for the excellent feedback! I've implemented exactly what you suggested. I added a raise_on_missing_reader flag to both LoadImage and LoadImaged classes that defaults to False to maintain...
Hi @ericspod thanks for the feedback! I've addressed both issues you mentioned: Fixed the guarding mechanism: Both `raise OptionalImportError` statements now use the same `raise_on_missing_reader` flag mechanism for consistent behavior....
Hey @ericspod Thank you for the code review feedback! I've addressed both of your suggestions in the latest commits: Refactored duplicate error messaging: • Lines 217-225: Extracted the common message...
> @dzenanz No that's fine I'll handle it, thanks for the reminder!
PR Updated - Ready for Review! Added raise_on_missing_reader flag to LoadImage/LoadImaged. All CodeRabbit feedback addressed, tests passing, backward compatible. Defaults to False for graceful fallback behavior.