added model_CT#8
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates dependencies and refines the CT model script by removing the SimpleITK dependency, skipping hidden files during prediction, and correcting the prediction output path for 3D predictions.
Changes:
- Added a
requirements.txtspecifying pinned versions for core scientific and ML libraries (NumPy, SciPy, TensorFlow, Keras, etc.). - Removed the unused
SimpleITKimport and cleaned up thescipy.ndimage.interpolation.zoomimport inmodel_CT_SS.py. - Updated
PredictandPredict3Dto skip hidden files in the test image directory and adjusted the 3D prediction save path to use the configuredsave_folderdirectly.
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| requirements.txt | Introduces explicit, pinned dependencies for the project’s Python ML stack, clarifying the environment needed to run the CT model. |
| model_CT_SS.py | Removes the unneeded SimpleITK import, ensures directory iteration skips hidden entries, and fixes the 3D prediction output path to align with how save_folder is configured in unet_CT_SS.py. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| print('case: ', each); | ||
| if each.startswith('.'): | ||
| continue | ||
| continue |
There was a problem hiding this comment.
There are two consecutive continue statements in this block, so the second one at the same indentation level is unreachable and will never execute. Please remove the redundant continue (or replace it with any missing logic you intended here) to avoid dead code and keep the control flow clear.
| continue |
Don't need ITK