Skip to content

Conversation

@bdhirsh
Copy link
Contributor

@bdhirsh bdhirsh commented Mar 28, 2022

Stack from ghstack (oldest at bottom):

Original commit changeset: 6d1dd0fd8144

Original Phabricator Diff: D34227616

Differential Revision: D35192346

NOTE FOR REVIEWERS: This PR has internal Facebook specific changes or comments, please review them on Phabricator!

Original commit changeset: 6d1dd0fd8144

Original Phabricator Diff: D34227616

Differential Revision: [D35192346](https://our.internmc.facebook.com/intern/diff/D35192346/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D35192346/)!

[ghstack-poisoned]
@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Mar 28, 2022

🔗 Helpful links

💊 CI failures summary and remediations

As of commit 19e59e2 (more details on the Dr. CI page):


  • 3/3 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 pull / linux-docs / build-docs (python) (1/1)

Step: "Build python docs" (full log | diagnosis details | 🔁 rerun)

2022-03-28T20:47:39.0638821Z Makefile:38: recipe for target 'html' failed
2022-03-28T20:47:37.5501583Z copying images... [ 92%] _static/img/tensorboard/add_histogram.png
2022-03-28T20:47:37.5503288Z copying images... [ 95%] _static/img/tensorboard/add_image.png
2022-03-28T20:47:37.5505054Z copying images... [ 97%] _static/img/tensorboard/add_images.png
2022-03-28T20:47:37.5507009Z copying images... [100%] _static/img/tensorboard/add_hparam.png
2022-03-28T20:47:37.5508743Z 
2022-03-28T20:47:37.5809025Z copying static files... done
2022-03-28T20:47:37.5809362Z copying extra files... done
2022-03-28T20:47:38.0293108Z dumping search index in English (code: en)... done
2022-03-28T20:47:38.1266409Z dumping object inventory... done
2022-03-28T20:47:38.1267089Z build finished with problems, 1 warning.
2022-03-28T20:47:39.0638821Z Makefile:38: recipe for target 'html' failed
2022-03-28T20:47:39.0639154Z make[1]: *** [html] Error 1
2022-03-28T20:47:39.0639633Z make[1]: Leaving directory '/var/lib/jenkins/workspace/docs'
2022-03-28T20:47:39.0640258Z Makefile:27: recipe for target 'html-stable' failed
2022-03-28T20:47:39.0640652Z make: *** [html-stable] Error 2
2022-03-28T20:47:39.0642371Z ++ code=2
2022-03-28T20:47:39.0642738Z ++ '[' 2 -ne 0 ']'
2022-03-28T20:47:39.0643621Z ++ set +x
2022-03-28T20:47:39.0643797Z =========================
2022-03-28T20:47:39.0711379Z /opt/conda/lib/python3.7/site-packages/torch/distributed/fsdp/fully_sharded_data_parallel.py:docstring of torch.distributed.fsdp.fully_sharded_data_parallel.FullyShardedDataParallel.state_dict_type:11: WARNING: Explicit markup ends without a blank line; unexpected unindent.
2022-03-28T20:47:39.0712196Z =========================

2 failures not recognized by patterns:

Job Step Action
GitHub Actions pull / linux-docs / build-docs (cpp) Unknown 🔁 rerun
GitHub Actions pull / linux-xenial-py3.7-gcc5.4 / test (docs_test, 1, 1, linux.2xlarge) Test 🔁 rerun

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.

bdhirsh added a commit that referenced this pull request Mar 28, 2022
Original commit changeset: 6d1dd0fd8144

Original Phabricator Diff: D34227616

Differential Revision: [D35192346](https://our.internmc.facebook.com/intern/diff/D35192346/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D35192346/)!

ghstack-source-id: 152381077
Pull Request resolved: #74859
facebook-github-bot pushed a commit to pytorch/nestedtensor that referenced this pull request Mar 29, 2022
Summary:
X-link: pytorch/pytorch#74859

Original commit changeset: 6d1dd0fd8144

Original Phabricator Diff: D34227616
ghstack-source-id: 152381077

(Note: this ignores all push blocking failures!)

Reviewed By: phding

Differential Revision: D35192346

fbshipit-source-id: b962de5d5effaf23f9aa8afd3ef36f8c6383de5b
facebook-github-bot pushed a commit that referenced this pull request Mar 29, 2022
Summary:
Pull Request resolved: #74859

Original commit changeset: 6d1dd0fd8144

Original Phabricator Diff: D34227616 (2cbddc0)
ghstack-source-id: 152381077

(Note: this ignores all push blocking failures!)

Test Plan:
Test on Milan with "get weather utterance"
buck build fbsourcefbandroid/mode/opt fbsourcefbandroid/mode/milan_build_rdk  //fbandroid/apps/wearable/system/speechservice:speechservice_target30_xhdpi_armv7_release_debug_keystore -c  pt.has_backtaces=1

Reviewed By: phding

Differential Revision: D35192346

fbshipit-source-id: b962de5d5effaf23f9aa8afd3ef36f8c6383de5b
@github-actions
Copy link
Contributor

Hey @bdhirsh.
You've committed this PR, but it does not have both a 'release notes: ...' and 'topics: ...' label. Please add one of each to the PR. The 'release notes: ...' label should represent the part of PyTorch that this PR changes (fx, autograd, distributed, etc) and the 'topics: ...' label should represent the kind of PR it is (not user facing, new feature, bug fix, perf improvement, etc). The list of valid labels can be found here for the 'release notes: ...' and here for the 'topics: ...'.
For changes that are 'topic: not user facing' there is no need for a release notes label.

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