-
Notifications
You must be signed in to change notification settings - Fork 737
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
110: Replace MO CLI with Python API and use PTQ nncf API #965
110: Replace MO CLI with Python API and use PTQ nncf API #965
Conversation
aleksandr-mokrov
commented
Mar 29, 2023
•
edited
edited
- MO CLI is replaced with Python API
- POT is replaced with PTQ nncf API.
- 110-ct-segmentation-quantize.ipynb is removed. Only 110-ct-segmentation-quantize-nncf.ipynb is actual.
- pytorch-monai-training.ipynb is updated: PyTorch Lightning has new incompatible interface (Remove memory-retaining epoch-end hooks Lightning-AI/pytorch-lightning#16520 for example), F1 score metric from POT replaced by BinaryF1Score from torchmetrics.
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
notebooks/110-ct-segmentation-quantize/110-ct-segmentation-quantize-nncf.ipynb
Show resolved
Hide resolved
notebooks/110-ct-segmentation-quantize/110-ct-segmentation-quantize-nncf.ipynb
Outdated
Show resolved
Hide resolved
… Usage of pytorch-lightning is changed due to new incompatible interface. 110-ct-segmentation-quantize.ipynb is deleted as outdated (replaced by 110-ct-segmentation-quantize-nncf.ipynb).
@@ -28,7 +28,7 @@ | |||
"\n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line #2. NIFTI_PATH = Path("~/PycharmProjects/default/kits19/data").expanduser()
please restore path
Reply via ReviewNB
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
@aleksandr-mokrov you have merge conflict, could you please resolve it? |
# Conflicts: # notebooks/110-ct-segmentation-quantize/110-ct-segmentation-quantize.ipynb
@aleksandr-mokrov please remove unused imports |
…lkit#965) * Removing yaspin from dependencies * F1 score metric from POT replaced by BinaryF1Score from torchmetrics. Usage of pytorch-lightning is changed due to new incompatible interface. 110-ct-segmentation-quantize.ipynb is deleted as outdated (replaced by 110-ct-segmentation-quantize-nncf.ipynb). * Improved description * Keep previous configuration * Return max_epochs to 1 if CPU * Return the default path. * Unused import is removed * Unused import is removed