[WIP] Adding SwinUNetR #12
Merged
Conversation
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
Codecov Report
@@ Coverage Diff @@
## main #12 +/- ##
==========================================
+ Coverage 37.34% 37.71% +0.37%
==========================================
Files 37 38 +1
Lines 3449 3500 +51
==========================================
+ Hits 1288 1320 +32
- Misses 2161 2180 +19
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
|
Preview page for your plugin is ready here: |
Member
|
@vidalmaxime can you fix merge conflicts? |
C-Achard
requested changes
Jul 13, 2022
Collaborator
C-Achard
left a comment
There was a problem hiding this comment.
Just a few comments on the PR @vidalmaxime :) Thanks for the work on the models
57d79ed to
b1be111
Compare
…xed padding only for sliding window inference
Contributor
Author
Removed draft of TRAILMAP port to Pytorch
- Fixed model instantiation - Fix window overlap argument type error - Updated reqs.txt to have einops (MONAI optional dep.)
- Added overlap parameter for window - Improved UI code slightly
Collaborator
|
Should be ready for review @vidalmaxime, if you could maybe test it quickly some time just to make sure everything is fine, that'd be great |
…xed padding only for sliding window inference
Removed draft of TRAILMAP port to Pytorch
- Fixed model instantiation - Fix window overlap argument type error - Updated reqs.txt to have einops (MONAI optional dep.)
- Added overlap parameter for window - Improved UI code slightly
da713e5 to
1030581
Compare
…ontrolLab/CellSeg3d into feature/swinunetr
Removed padding transform when using single layer inference and the sliding window inference option
Prepared for SWIN download
- Fixed error in training : get_net erroneous call
Collaborator
|
@vidalmaxime I think this is ready for merging into main - should we do a new rc on PyPi with the recently added features ? |
C-Achard
approved these changes
Aug 15, 2022
Member
|
yes, let's make a new pypi, so new PR with the version bumped up! |
MMathisLab
approved these changes
Aug 15, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
(WIP) This PR aims to
Add SwinUNetR to the modelzoo
Remove extra padding during inference which can worsen the predictions when using window inference
Removed old TRAILMAP copy