Skip to content

Drop radam optimizer.#377

Merged
jettify merged 1 commit intomasterfrom
drop-radam-optimizer
Oct 12, 2021
Merged

Drop radam optimizer.#377
jettify merged 1 commit intomasterfrom
drop-radam-optimizer

Conversation

@jettify
Copy link
Owner

@jettify jettify commented Oct 12, 2021

Drop radam since it is included in pytorch.

@codecov-commenter
Copy link

Codecov Report

Merging #377 (cde4b29) into master (2c8da6a) will decrease coverage by 0.80%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #377      +/-   ##
==========================================
- Coverage   83.50%   82.70%   -0.81%     
==========================================
  Files          27       26       -1     
  Lines        1716     1642      -74     
  Branches      391      371      -20     
==========================================
- Hits         1433     1358      -75     
- Misses        165      167       +2     
+ Partials      118      117       -1     
Impacted Files Coverage Δ
torch_optimizer/__init__.py 88.88% <ø> (-0.31%) ⬇️
torch_optimizer/sgdp.py 77.63% <0.00%> (-6.58%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2c8da6a...cde4b29. Read the comment docs.

@jettify jettify merged commit 4bb5e4b into master Oct 12, 2021
@jettify jettify deleted the drop-radam-optimizer branch October 19, 2021 03:28
ben-freist added a commit to ben-freist/asteroid that referenced this pull request Oct 28, 2021
In jettify/pytorch-optimizer#377 the RAdam optimizer was removed from
pytorch-optimizer, it's still used in aseroid though.
mpariente pushed a commit to asteroid-team/asteroid that referenced this pull request Oct 28, 2021
In jettify/pytorch-optimizer#377 the RAdam optimizer was removed from
pytorch-optimizer, it's still used in asteroid though.
@clayheaton
Copy link

It would have been nice to deprecate this more gracefully -- the change just broke most of the VQGAN+CLIP tutorials that are out in the wild.

@jettify
Copy link
Owner Author

jettify commented Oct 31, 2021

Sorry about that, will restore with proper deprecation.

@jettify jettify restored the drop-radam-optimizer branch October 31, 2021 02:49
jettify added a commit that referenced this pull request Oct 31, 2021
jettify added a commit that referenced this pull request Oct 31, 2021
@jettify
Copy link
Owner Author

jettify commented Oct 31, 2021

@clayheaton sorry for inconvenience I reverted change and release new version.
https://pypi.org/project/torch-optimizer/0.3.0/

@jettify jettify deleted the drop-radam-optimizer branch October 31, 2021 03:15
@clayheaton
Copy link

clayheaton commented Oct 31, 2021 via email

can-cui pushed a commit to can-cui/asteroid-related that referenced this pull request Jun 15, 2025
In jettify/pytorch-optimizer#377 the RAdam optimizer was removed from
pytorch-optimizer, it's still used in asteroid though.
hedonemaeda pushed a commit to hedonemaeda/pytorch-optimizer that referenced this pull request Mar 15, 2026
hedonemaeda pushed a commit to hedonemaeda/pytorch-optimizer that referenced this pull request Mar 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants