[profiler][small] Add skip_first parameter to the default schedule#58025

Closed

ilia-cher wants to merge 9 commits intogh/ilia-cher/107/basefrom

gh/ilia-cher/107/head

Contributor

ilia-cher commented May 11, 2021 •

edited

Loading

Stack from ghstack:

[profiler][small] Add skip_first parameter to the default schedule #58025 [profiler][small] Add skip_first parameter to the default schedule
[profiler] Speed up postprocessing #58021 [profiler][small] Speed up postprocessing
[profiler][small] Handle empty trace #58013 [profiler][small] Handle empty trace
[profiler][resend] Add cuda memory and distributed metadata #58010 [profiler][resend] Add cuda memory and distributed metadata

Summary:
Add skip_first parameter to allow for arbitrary profiler step ranges

Test Plan:
python test/test_profiler.py

Differential Revision: D28347768


          [profiler][small] Add skip_first parameter to the default schedule

1f6408b

Summary:
Add `skip_first` parameter to allow for arbitrary profiler step ranges

Test Plan:
python test/test_profiler.py

[ghstack-poisoned]

This was referenced May 11, 2021

[profiler][resend] Add cuda memory and distributed metadata #58010

Closed

[profiler][small] Handle empty trace #58013

Closed

[profiler] Speed up postprocessing #58021

Closed

facebook-github-bot added the cla signed label

Contributor

facebook-github-bot commented May 11, 2021 •

edited

Loading

💊 CI failures summary and remediations

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

1/1 failures possibly* introduced in this PR
- 1/1 non-scanned failure(s)

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

Follow this link to opt-out of these comments for your Pull Requests.

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

Click here to manually regenerate this comment.

ilia-cher pushed a commit that referenced this pull request


          [profiler][small] Add skip_first parameter to the default schedule

cb5069a

Summary:
Add `skip_first` parameter to allow for arbitrary profiler step ranges

Test Plan:
python test/test_profiler.py

ghstack-source-id: 800c63f
Pull Request resolved: #58025

Contributor Author

ilia-cher commented May 11, 2021

@ilia-cher has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.


          Update on "[profiler][small] Add skip_first parameter to the default …

2ca12ac

…schedule"

Summary:
Add `skip_first` parameter to allow for arbitrary profiler step ranges

Test Plan:
python test/test_profiler.py

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

[ghstack-poisoned]

Contributor Author

ilia-cher commented May 11, 2021

@ilia-cher has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.


          Update on "[profiler][small] Add skip_first parameter to the default …

92c60b4

…schedule"

Summary:
Add `skip_first` parameter to allow for arbitrary profiler step ranges

Test Plan:
python test/test_profiler.py

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

[ghstack-poisoned]

ilia-cher pushed a commit that referenced this pull request


          [profiler][small] Add skip_first parameter to the default schedule

ed04f68

Summary:
Add `skip_first` parameter to allow for arbitrary profiler step ranges

Test Plan:
python test/test_profiler.py

ghstack-source-id: 5ac3fde
Pull Request resolved: #58025

Contributor Author

ilia-cher commented May 11, 2021

@ilia-cher has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

ilia-cher requested review from bitfort, chaekit, gdankel and xuzhao9

May 11, 2021 08:34


          Update on "[profiler][small] Add skip_first parameter to the default …

9b57620

…schedule"

Summary:
Add `skip_first` parameter to allow for arbitrary profiler step ranges

Test Plan:
python test/test_profiler.py

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

[ghstack-poisoned]

ilia-cher pushed a commit that referenced this pull request


          [profiler][small] Add skip_first parameter to the default schedule

a72aaf3

Summary:
Add `skip_first` parameter to allow for arbitrary profiler step ranges

Test Plan:
python test/test_profiler.py

ghstack-source-id: ebf4aa0
Pull Request resolved: #58025

Contributor Author

ilia-cher commented May 11, 2021

@ilia-cher has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.


          Update on "[profiler][small] Add skip_first parameter to the default …

08128fa

…schedule"

Summary:
Add `skip_first` parameter to allow for arbitrary profiler step ranges

Test Plan:
python test/test_profiler.py

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

[ghstack-poisoned]

Contributor Author

ilia-cher commented May 11, 2021

@ilia-cher has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.


          Update on "[profiler][small] Add skip_first parameter to the default …

cd20477

…schedule"

Summary:
Add `skip_first` parameter to allow for arbitrary profiler step ranges

Test Plan:
python test/test_profiler.py

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

[ghstack-poisoned]

ilia-cher pushed a commit that referenced this pull request


          [profiler][small] Add skip_first parameter to the default schedule

721b1ff

Summary:
Add `skip_first` parameter to allow for arbitrary profiler step ranges

Test Plan:
python test/test_profiler.py

ghstack-source-id: cc93dc7
Pull Request resolved: #58025

Contributor Author

ilia-cher commented May 11, 2021

@ilia-cher has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.


          Update on "[profiler][small] Add skip_first parameter to the default …

6248c80

…schedule"

Summary:
Add `skip_first` parameter to allow for arbitrary profiler step ranges

Test Plan:
python test/test_profiler.py

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

[ghstack-poisoned]

Contributor Author

ilia-cher commented May 12, 2021

@ilia-cher has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.


          Update on "[profiler][small] Add skip_first parameter to the default …

cd4ec3d

…schedule"

Summary:
Add `skip_first` parameter to allow for arbitrary profiler step ranges

Test Plan:
python test/test_profiler.py

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

[ghstack-poisoned]

Contributor Author

ilia-cher commented May 12, 2021

@ilia-cher has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.


          Update on "[profiler][small] Add skip_first parameter to the default …

436ff70

…schedule"

Summary:
Add `skip_first` parameter to allow for arbitrary profiler step ranges

Test Plan:
python test/test_profiler.py

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

[ghstack-poisoned]

ilia-cher pushed a commit that referenced this pull request


          [profiler][small] Add skip_first parameter to the default schedule

99c8c58

Summary:
Add `skip_first` parameter to allow for arbitrary profiler step ranges

Test Plan:
python test/test_profiler.py

ghstack-source-id: 5018a5f
Pull Request resolved: #58025

Contributor Author

ilia-cher commented May 12, 2021

@ilia-cher has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

facebook-github-bot closed this in

e18f5f1

Contributor

facebook-github-bot commented May 12, 2021

@ilia-cher merged this pull request in e18f5f1.

facebook-github-bot added the Merged label

facebook-github-bot deleted the gh/ilia-cher/107/head branch

May 15, 2021 14:23

krshrimali pushed a commit to krshrimali/pytorch that referenced this pull request


          [profiler][small] Add skip_first parameter to the default schedule (p…

55500fe

…ytorch#58025)

Summary:
Pull Request resolved: pytorch#58025

Add `skip_first` parameter to allow for arbitrary profiler step ranges

Test Plan: python test/test_profiler.py

Reviewed By: gdankel

Differential Revision: D28347768

Pulled By: ilia-cher

fbshipit-source-id: bb6fd3cedfa4a5d1307b91002def733896dd03eb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed Merged