Skip to content
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

Drop radam optimizer. #377

Merged
merged 1 commit into from
Oct 12, 2021
Merged

Drop radam optimizer. #377

merged 1 commit into from
Oct 12, 2021

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

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.

None yet

3 participants