Open Milad4849 opened 11 months ago
The recent push includes change and fixes to address all the issues raised. I will now move on to test the Docker containers. After this PR is merged I will move on to address other issues via feature branches.
Hi @Jonas, how should we proceed with this PR?
Hi both, sorry for not getting back to you earlier. As commented, I've only reviewed the documentation-related issues so far. Will now take a closer look at the code changes and get back to you asap.
Hi @Milad4849! Apologies for the delay, I completely missed that you re-requested a review. Will do it asap, latest at the end of this week.
CC @SchulzDan
Also, the code still contains #TODO
for list-like options
Cellpose
added as an app (steinbock/_cli/apps.py
).Cellpose
model training added (steinbock/segmentation/_cli/train.py', 'steinbock/segmentation/train.py
, 'Steinbock/segmentation/_cli/_cli.py').Cellpose
default parameters changed (steinbock/segmentation/cellpose.py
).Cellpose
model zoo added (steinbock/segmentation/_cli/cellpose.py
).CONTRIBUTORS.md
).CHANGELOG.md
updated.docs/cli/segmentation.md
).docs/cli/utils.md
).docs/install-docker.md
).Opencv-python==4.8.0.74
added to requirements, it was needed to run cellpose training (requirements.txt
).steinbock/preprocessing/imc.py
).