[Fix] Add setr & vit msg. #635
Merged
Conversation
This file contains 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
Junjun2016
reviewed
Jun 23, 2021
| @@ -25,6 +25,7 @@ def __init__(self, | |||
| num_convs=1, | |||
| up_scale=4, | |||
| kernel_size=3, | |||
| init_cfg=None, | |||
There was a problem hiding this comment.
We can set the default init_cfg to implement the same weights initialization as that in init_weights
There was a problem hiding this comment.
And we can set it to None in configs if SETR does not use it.
Codecov Report
@@ Coverage Diff @@
## master #635 +/- ##
==========================================
- Coverage 85.78% 85.77% -0.02%
==========================================
Files 103 103
Lines 5312 5307 -5
Branches 859 857 -2
==========================================
- Hits 4557 4552 -5
Misses 583 583
Partials 172 172
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
added 2 commits
June 24, 2021 10:55
xvjiarui
approved these changes
Jun 24, 2021
bowenroom
pushed a commit
to bowenroom/mmsegmentation
that referenced
this pull request
Feb 25, 2022
* [Fix] Add setr & vit msg. * Fix init bug * Modify init_cfg arg * Add conv_seg init
aravind-h-v
pushed a commit
to aravind-h-v/mmsegmentation
that referenced
this pull request
Mar 27, 2023
* correcting the beta value assignment * updating DDIM and LMSDiscreteFlax schedulers * bringing back the changes that were lost as part of main branch merge
wjkim81
pushed a commit
to wjkim81/mmsegmentation
that referenced
this pull request
Dec 3, 2023
* fix opencv-python 4.5.2 compatibility BottomUpRandomAffine fails with opencv-python>=4.5.2 because a single valued numpy.ndarray is no longer a valid data type for image_size in cv2.warpAffine. This commit fix it by repacing image_size with image_size.item(). * fix smplx 0.1.27 compatibility smplx>=0.1.27 requires loguru, which will not be automatically installed when installing smplx. So we add loguru into mmpose requirements.
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.
Motivation
The lack of SETR & ViT in the model list.
The SETRUPHead still use deprecated init_weights.
Modification
Add SETR & ViT url in the main readme doc.
Refactor init of SETRUPHead.