Skip to content

Conversation

xiaohu2015
Copy link
Contributor

@xiaohu2015 xiaohu2015 commented Nov 12, 2021

The pr is about #4509
when use amp, it is better to save/resuem scaler state_dict https://pytorch.org/tutorials/recipes/recipes/amp_recipe.html#saving-resuming

cc @datumbox

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Nov 12, 2021

💊 CI failures summary and remediations

As of commit 42c1c4f (more details on the Dr. CI page):


  • 1/1 failures introduced in this PR

🕵️ 1 new failure recognized by patterns

The following CI failures do not appear to be due to upstream breakages:

See GitHub Actions build CodeQL / build (1/1)

Step: "Perform CodeQL Analysis" (full log | diagnosis details | 🔁 rerun)

2021-11-12T09:56:10.6351445Z ##[error]source repository for pull request could not be determined
2021-11-12T09:56:09.8303807Z TRAP files cleaned up (3ms).
2021-11-12T09:56:09.8308131Z Cleaning up scratch directory...
2021-11-12T09:56:09.8326612Z Scratch directory cleaned up (0ms).
2021-11-12T09:56:09.8327347Z Cleaning up temporary directory...
2021-11-12T09:56:09.8327943Z Temporary directory cleaned up (0ms).
2021-11-12T09:56:09.9465461Z ##[endgroup]
2021-11-12T09:56:09.9465789Z 
2021-11-12T09:56:09.9701760Z ##[group]Uploading results
2021-11-12T09:56:09.9702710Z Processing sarif files: ["/home/runner/work/vision/results/cpp.sarif","/home/runner/work/vision/results/python.sarif"]
2021-11-12T09:56:10.2983063Z Uploading results
2021-11-12T09:56:10.6351445Z ##[error]source repository for pull request could not be determined
2021-11-12T09:56:10.6383729Z RequestError [HttpError]: source repository for pull request could not be determined
2021-11-12T09:56:10.6385698Z     at /home/runner/work/_actions/github/codeql-action/v1/node_modules/@octokit/request/dist-node/index.js:66:23
2021-11-12T09:56:10.6386590Z     at runMicrotasks (<anonymous>)
2021-11-12T09:56:10.6387457Z     at processTicksAndRejections (internal/process/task_queues.js:93:5)
2021-11-12T09:56:10.6388883Z     at async Job.doExecute (/home/runner/work/_actions/github/codeql-action/v1/node_modules/bottleneck/light.js:405:18) {
2021-11-12T09:56:10.6389869Z   name: 'HttpError',
2021-11-12T09:56:10.6390281Z   status: 422,
2021-11-12T09:56:10.6390675Z   headers: {
2021-11-12T09:56:10.6391439Z     'access-control-allow-origin': '*',
2021-11-12T09:56:10.6394225Z     'access-control-expose-headers': 'ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Resource, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, Deprecation, Sunset',

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.

Copy link
Contributor

@datumbox datumbox left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@datumbox datumbox merged commit 22ff44f into pytorch:main Nov 12, 2021
cyyever pushed a commit to cyyever/vision that referenced this pull request Nov 16, 2021
Co-authored-by: Vasilis Vryniotis <datumbox@users.noreply.github.com>
facebook-github-bot pushed a commit that referenced this pull request Nov 17, 2021
Summary: Co-authored-by: Vasilis Vryniotis <datumbox@users.noreply.github.com>

Reviewed By: datumbox

Differential Revision: D32470472

fbshipit-source-id: bf8467cadba71e73ceaac5993162dd122476492c
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