-
Notifications
You must be signed in to change notification settings - Fork 16
feat: enhance prediction handling by validating output channels and a… #188
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
Merged
Merged
Changes from all commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
3732322
feat: enhance prediction handling by validating output channels and a…
rhoadesScholar 3de888c
Merge branch 'main' into multi_channel_writing
rhoadesScholar e0961bf
Update src/cellmap_segmentation_challenge/predict.py
rhoadesScholar 8467547
Update src/cellmap_segmentation_challenge/predict.py
rhoadesScholar 5329c81
fix: adjust shape check for target arrays in _predict function
rhoadesScholar 4ca53a7
feat: enhance model input/output handling with singleton dimension ut…
rhoadesScholar 04cbe5f
Initial plan
Copilot 5da027f
Initial plan
Copilot f259b36
Update tests/test_utils.py
rhoadesScholar f6b6578
fix: use rank-based check for channel dimension detection
Copilot d998f2d
Merge branch 'multi_channel_writing' into copilot/sub-pr-188-another-one
rhoadesScholar b8595cf
Merge branch 'multi_channel_writing' into copilot/sub-pr-188
rhoadesScholar 84aa23d
feat: add validation for num_channels_per_class in structure_model_ou…
Copilot 6915174
fix: remove duplicate TestDownloadFile class and format code
Copilot 6696cf1
refactor: improve test method names for clarity
Copilot 1e18d67
feat: use deepcopy and spatial rank check for robust shape handling
Copilot c15980d
test: add comprehensive unit tests for shape adjustment logic
Copilot 99b75ed
refactor: address code review feedback on imports and clarity
Copilot a2e8902
Merge pull request #193 from janelia-cellmap/copilot/sub-pr-188-anoth…
rhoadesScholar 619a23c
Merge branch 'multi_channel_writing' into copilot/sub-pr-188
rhoadesScholar 9595f57
Merge pull request #191 from janelia-cellmap/copilot/sub-pr-188
rhoadesScholar 4bc7b2e
Apply suggestion from @rhoadesScholar
rhoadesScholar 7b68a2f
Apply suggestion from @Copilot
rhoadesScholar 102bb09
Apply suggestion from @rhoadesScholar
rhoadesScholar 4865a59
fix: improve documentation in predict_2D.py and predict_3D.py
rhoadesScholar dcbb99c
fix: update cellmap-data dependency version to >=2026.2.19.2140
rhoadesScholar d315e4b
fix: adjust shape handling in prediction functions and improve deepco…
rhoadesScholar fbfdd06
Merge branch 'main' into multi_channel_writing
rhoadesScholar 8757c23
Update src/cellmap_segmentation_challenge/predict.py
rhoadesScholar c1f9c36
Apply suggestion from @rhoadesScholar
rhoadesScholar b7878f1
Initial plan
Copilot 4acf5fa
fix: treat num_channels_per_class=1 same as None to match docstring b…
Copilot 248caf4
Merge pull request #196 from janelia-cellmap/copilot/sub-pr-188
rhoadesScholar File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.