Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Log the optimus graph transformation to the scuba #2163

Closed
wants to merge 1 commit into from

Conversation

mengluy0125
Copy link
Contributor

Summary: Current everstore upload logging may cuase excessive compilation time when the model has lots of graph breaks (post: https://fb.workplace.com/groups/257735836456307/permalink/633533465543207/), we here log the transformation only when the graph changed

Differential Revision: D53692344

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D53692344

mengluy0125 added a commit to mengluy0125/benchmark that referenced this pull request Feb 15, 2024
Summary:

X-link: pytorch/pytorch#119745

Current everstore upload logging may cuase excessive compilation time when the model has lots of graph breaks (post: https://fb.workplace.com/groups/257735836456307/permalink/633533465543207/), we here log the transformation only when the graph changed

Reviewed By: jackiexu1992

Differential Revision: D53692344
mengluy0125 added a commit to mengluy0125/pytorch that referenced this pull request Feb 15, 2024
… the scuba (pytorch#119745)

Summary:
X-link: pytorch/benchmark#2163


Current everstore upload logging may cuase excessive compilation time when the model has lots of graph breaks (post: https://fb.workplace.com/groups/257735836456307/permalink/633533465543207/), we here log the transformation only when the graph changed

Test Plan:
# unit test
```
buck2 test //caffe2/test/inductor:group_batch_fusion
```
Buck UI: https://www.internalfb.com/buck2/8dc77120-76ea-41bf-83b4-ada88bf1e2c8
Test UI: https://www.internalfb.com/intern/testinfra/testrun/7036874622840948
Network: Up: 321KiB  Down: 33MiB  (reSessionID-5372a58f-8b66-43a9-82b4-ee23aeaa44b9)
Jobs completed: 20. Time elapsed: 4:13.4s.
Cache hits: 0%. Commands: 2 (cached: 0, remote: 0, local: 2)
Tests finished: Pass 7. Fail 0. Fatal 0. Skip 0. Build failure 0

```
buck2 test //caffe2/test/inductor:split_cat_fx_passes
```
Buck UI: https://www.internalfb.com/buck2/abc8b8f8-d240-47d3-ad9d-cae13b8e62d3
Test UI: https://www.internalfb.com/intern/testinfra/testrun/13229323926967362
Network: Up: 119KiB  Down: 66KiB  (reSessionID-8bdd34b3-159b-469f-9f00-8384620c13ea)
Jobs completed: 28. Time elapsed: 3:02.9s.
Cache hits: 0%. Commands: 2 (cached: 0, remote: 0, local: 2)
Tests finished: Pass 11. Fail 0. Fatal 0. Skip 0. Build failure 0



# e2e
baseline:
f528209775
proposal:
f531285723

scuba: https://fburl.com/scuba/workflow_signpost/7hamzr64

 {F1456548774}

Reviewed By: jackiexu1992

Differential Revision: D53692344
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D53692344

mengluy0125 added a commit to mengluy0125/benchmark that referenced this pull request Feb 15, 2024
Summary:

X-link: pytorch/pytorch#119745

Current everstore upload logging may cuase excessive compilation time when the model has lots of graph breaks (post: https://fb.workplace.com/groups/257735836456307/permalink/633533465543207/), we here log the transformation only when the graph changed

Reviewed By: jackiexu1992

Differential Revision: D53692344
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D53692344

Summary:

X-link: pytorch/pytorch#119745

Current everstore upload logging may cuase excessive compilation time when the model has lots of graph breaks (post: https://fb.workplace.com/groups/257735836456307/permalink/633533465543207/), we here log the transformation only when the graph changed

Reviewed By: jackiexu1992

Differential Revision: D53692344
mengluy0125 added a commit to mengluy0125/pytorch that referenced this pull request Feb 15, 2024
… the scuba (pytorch#119745)

Summary:
X-link: pytorch/benchmark#2163


Current everstore upload logging may cuase excessive compilation time when the model has lots of graph breaks (post: https://fb.workplace.com/groups/257735836456307/permalink/633533465543207/), we here log the transformation only when the graph changed

Test Plan:
# unit test
```
buck2 test //caffe2/test/inductor:group_batch_fusion
```
Buck UI: https://www.internalfb.com/buck2/8dc77120-76ea-41bf-83b4-ada88bf1e2c8
Test UI: https://www.internalfb.com/intern/testinfra/testrun/7036874622840948
Network: Up: 321KiB  Down: 33MiB  (reSessionID-5372a58f-8b66-43a9-82b4-ee23aeaa44b9)
Jobs completed: 20. Time elapsed: 4:13.4s.
Cache hits: 0%. Commands: 2 (cached: 0, remote: 0, local: 2)
Tests finished: Pass 7. Fail 0. Fatal 0. Skip 0. Build failure 0

```
buck2 test //caffe2/test/inductor:split_cat_fx_passes
```
Buck UI: https://www.internalfb.com/buck2/abc8b8f8-d240-47d3-ad9d-cae13b8e62d3
Test UI: https://www.internalfb.com/intern/testinfra/testrun/13229323926967362
Network: Up: 119KiB  Down: 66KiB  (reSessionID-8bdd34b3-159b-469f-9f00-8384620c13ea)
Jobs completed: 28. Time elapsed: 3:02.9s.
Cache hits: 0%. Commands: 2 (cached: 0, remote: 0, local: 2)
Tests finished: Pass 11. Fail 0. Fatal 0. Skip 0. Build failure 0



# e2e
baseline:
f528209775
proposal:
f531285723

scuba: https://fburl.com/scuba/workflow_signpost/7hamzr64

 {F1456548774}

Reviewed By: jackiexu1992

Differential Revision: D53692344
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D53692344

mengluy0125 added a commit to mengluy0125/pytorch that referenced this pull request Feb 15, 2024
… the scuba (pytorch#119745)

Summary:
X-link: pytorch/benchmark#2163


Current everstore upload logging may cuase excessive compilation time when the model has lots of graph breaks (post: https://fb.workplace.com/groups/257735836456307/permalink/633533465543207/), we here log the transformation only when the graph changed

Test Plan:
# unit test
```
buck2 test //caffe2/test/inductor:group_batch_fusion
```
Buck UI: https://www.internalfb.com/buck2/8dc77120-76ea-41bf-83b4-ada88bf1e2c8
Test UI: https://www.internalfb.com/intern/testinfra/testrun/7036874622840948
Network: Up: 321KiB  Down: 33MiB  (reSessionID-5372a58f-8b66-43a9-82b4-ee23aeaa44b9)
Jobs completed: 20. Time elapsed: 4:13.4s.
Cache hits: 0%. Commands: 2 (cached: 0, remote: 0, local: 2)
Tests finished: Pass 7. Fail 0. Fatal 0. Skip 0. Build failure 0

```
buck2 test //caffe2/test/inductor:split_cat_fx_passes
```
Buck UI: https://www.internalfb.com/buck2/abc8b8f8-d240-47d3-ad9d-cae13b8e62d3
Test UI: https://www.internalfb.com/intern/testinfra/testrun/13229323926967362
Network: Up: 119KiB  Down: 66KiB  (reSessionID-8bdd34b3-159b-469f-9f00-8384620c13ea)
Jobs completed: 28. Time elapsed: 3:02.9s.
Cache hits: 0%. Commands: 2 (cached: 0, remote: 0, local: 2)
Tests finished: Pass 11. Fail 0. Fatal 0. Skip 0. Build failure 0



# e2e
baseline:
f528209775
proposal:
f531285723

scuba: https://fburl.com/scuba/workflow_signpost/7hamzr64

 {F1456548774}

Reviewed By: jackiexu1992

Differential Revision: D53692344
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in f5b2e50.

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