Skip to content

Enhancement for WarmupCosineSchedule to specify the beginning fraction of the linear warmup#5351

Merged
wyli merged 2 commits intoProject-MONAI:devfrom
myron:warmup
Oct 18, 2022
Merged

Enhancement for WarmupCosineSchedule to specify the beginning fraction of the linear warmup#5351
wyli merged 2 commits intoProject-MONAI:devfrom
myron:warmup

Conversation

@myron
Copy link
Copy Markdown
Collaborator

@myron myron commented Oct 17, 2022

small enhancement to WarmupCosineSchedule input, to optionally specify the beginning of the linear warmup from something above 0 ( e.g from a fraction 0.1 * initial_lr).

a unit test is added too.

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • Breaking change (fix or new feature that would cause existing functionality to change).
  • New tests added to cover the changes.
  • Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
  • Quick tests passed locally by running ./runtests.sh --quick --unittests --disttests.
  • In-line docstrings updated.
  • Documentation updated, tested make html command in the docs/ folder.

@myron myron added the enhancement New feature or request label Oct 17, 2022
@myron myron added this to the Auto3D Seg framework [P0 v1.0] milestone Oct 17, 2022
Signed-off-by: myron <[email protected]>
@wyli
Copy link
Copy Markdown
Contributor

wyli commented Oct 18, 2022

looks like this option will address #3989, could you confirm?

@wyli
Copy link
Copy Markdown
Contributor

wyli commented Oct 18, 2022

/build

@myron
Copy link
Copy Markdown
Collaborator Author

myron commented Oct 18, 2022

looks like this option will address #3989, could you confirm?

@wyli @holgerroth
Yes, I looked at it, and it will solve the issue in #3989.
It's to prevent the same situation when the first iterations to be zero (or almost zero) during linear warmup.
The change is non-breaking, and the code will work as before, unless a user specifies this option. I don't think we need a whole new Scheduler for such a small change.

@wyli wyli merged commit 4bbf125 into Project-MONAI:dev Oct 18, 2022
wyli pushed a commit that referenced this pull request Oct 18, 2022
…n of the linear warmup (#5351)

small enhancement to WarmupCosineSchedule input, to optionally specify
the beginning of the linear warmup from something above 0 ( e.g from a
fraction 0.1 * initial_lr).

a unit test is added too.


### Types of changes
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [x] Non-breaking change (fix or new feature that would not break
existing functionality).
- [ ] Breaking change (fix or new feature that would cause existing
functionality to change).
- [x] New tests added to cover the changes.
- [x] Integration tests passed locally by running `./runtests.sh -f -u
--net --coverage`.
- [x] Quick tests passed locally by running `./runtests.sh --quick
--unittests --disttests`.
- [x] In-line docstrings updated.
- [ ] Documentation updated, tested `make html` command in the `docs/`
folder.

Signed-off-by: myron <[email protected]>
@myron myron deleted the warmup branch October 22, 2022 18:50
KumoLiu pushed a commit that referenced this pull request Nov 2, 2022
…n of the linear warmup (#5351)

small enhancement to WarmupCosineSchedule input, to optionally specify
the beginning of the linear warmup from something above 0 ( e.g from a
fraction 0.1 * initial_lr).

a unit test is added too.


### Types of changes
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [x] Non-breaking change (fix or new feature that would not break
existing functionality).
- [ ] Breaking change (fix or new feature that would cause existing
functionality to change).
- [x] New tests added to cover the changes.
- [x] Integration tests passed locally by running `./runtests.sh -f -u
--net --coverage`.
- [x] Quick tests passed locally by running `./runtests.sh --quick
--unittests --disttests`.
- [x] In-line docstrings updated.
- [ ] Documentation updated, tested `make html` command in the `docs/`
folder.

Signed-off-by: myron <[email protected]>
Signed-off-by: KumoLiu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants