Skip to content

Conversation

@mikaylagawarecki
Copy link
Contributor

@mikaylagawarecki mikaylagawarecki commented Apr 4, 2022

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Apr 4, 2022

🔗 Helpful links

💊 CI failures summary and remediations

As of commit 41225a7 (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

@mikaylagawarecki mikaylagawarecki changed the title Fix casting bug in state_step for optimizers when loading state dict Fix casting bug for state['step'] of optimizers when loading state dict Apr 4, 2022
mikaylagawarecki added a commit that referenced this pull request Apr 4, 2022
Copy link
Collaborator

@albanD albanD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From offline discussion.
This is an ok localized fix for the most pressing matter. We will open an issue an investigate a better general solution.

@mikaylagawarecki
Copy link
Contributor Author

@pytorchbot merge this please

facebook-github-bot pushed a commit that referenced this pull request Apr 6, 2022
…75214)

Summary:
Pull Request resolved: #75214

Approved by: https://github.com/albanD

Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/10bb0ffe698b51ce8f11fa0ab3870bac10d96072

Reviewed By: b0noI

Differential Revision: D35404323

Pulled By: mikaylagawarecki

fbshipit-source-id: ea797028bd67b535d607eb0b3ddf946685037263
@facebook-github-bot facebook-github-bot deleted the gh/mikaylagawarecki/51/head branch April 8, 2022 14:17
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.

4 participants