-
Notifications
You must be signed in to change notification settings - Fork 0
Update AICSImageProcessing #4
Description
Use Case
aicsimageprocessing is used heavily in the single cell processing pipeline, and other already released projects (segmenter), and it isn't in the greatest place. Both for my sanity, and for cleaning code standards it would be great to update the package to have tests for every function, add docstrings, etc.
Solution
Not necessarily a full rewrite of the package but something in-between at least docstring upgrades and a full rewrite would be great. I am specifically thinking about how I believe some functions are written specifically with image dimensions assumed when passed in but not documented anywhere.
Alternatives
Leave as is. Continue on with added tech debt.
Stakeholders
Assay Dev (Matheus, Jianxu), Modeling (Rory, Jackson), and Dan
Major Components
[ ] Initial work of cataloging issues or potential problem spots with stakeholders.
[ ] Plan for stable API
[ ] Implement major revisions
[ ] Implement tests + docstrings
Dependencies
None