Skip to content

[ci/docs] add some documentation about the stats uploading process#79504

Closed
suo wants to merge 2 commits intogh/suo/569/basefrom
gh/suo/569/head
Closed

[ci/docs] add some documentation about the stats uploading process#79504
suo wants to merge 2 commits intogh/suo/569/basefrom
gh/suo/569/head

Conversation

@suo
Copy link
Copy Markdown
Member

@suo suo commented Jun 14, 2022

Stack from ghstack (oldest at bottom):

This process is pretty confusing, so wrote it down.

[skip ci]

This process is pretty confusing, so wrote it down.

[skip ci]

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

facebook-github-bot commented Jun 14, 2022

🔗 Helpful links

✅ No Failures (0 Pending)

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

Expand to see more

💚 💚 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.

suo added a commit that referenced this pull request Jun 14, 2022
This process is pretty confusing, so wrote it down.

[skip ci]

ghstack-source-id: b16f0eb
Pull Request resolved: #79504
@suo suo requested a review from janeyx99 June 14, 2022 06:18
Comment thread tools/stats/README.md Outdated
https://github.com/pytorch/pytorch/blob/a9f6a35a33308f3be2413cc5c866baec5cfe3ba1/.github/workflows/_linux-build.yml#L144-L151
2. When a workflow completes, a `workflow_run` event [triggers
`upload-test-stats.yml`](https://github.com/pytorch/pytorch/blob/d9fca126fca7d7780ae44170d30bda901f4fe35e/.github/workflows/upload-test-stats.yml#L4).
3. `upload-test-stats` downloads the raw stats form the intermediate data store
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
3. `upload-test-stats` downloads the raw stats form the intermediate data store
3. `upload-test-stats` downloads the raw stats from the intermediate data store

Copy link
Copy Markdown
Contributor

@janeyx99 janeyx99 left a comment

Choose a reason for hiding this comment

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

yay

… process"

This process is pretty confusing, so wrote it down.

[skip ci]

[ghstack-poisoned]
suo added a commit that referenced this pull request Jun 14, 2022
This process is pretty confusing, so wrote it down.

[skip ci]

ghstack-source-id: d5c54c7
Pull Request resolved: #79504
@suo
Copy link
Copy Markdown
Member Author

suo commented Jun 14, 2022

@pytorchbot merge -f

@pytorchmergebot
Copy link
Copy Markdown
Collaborator

@pytorchbot successfully started a merge job. Check the current status here

facebook-github-bot pushed a commit that referenced this pull request Jun 16, 2022
…79504)

Summary:
This process is pretty confusing, so wrote it down.

[skip ci]

Pull Request resolved: #79504

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

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

Reviewed By: malfet

Differential Revision: D37157003

Pulled By: suo

fbshipit-source-id: ec322893fe3b8f44cd0412d27aa2a03db2de4dac
@facebook-github-bot facebook-github-bot deleted the gh/suo/569/head branch June 18, 2022 14:17
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 25, 2026
This process is pretty confusing, so wrote it down.

[skip ci]

Pull Request resolved: pytorch#79504

Approved by: https://github.com/janeyx99
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