Skip to content

Conversation

@vkuzo
Copy link
Contributor

@vkuzo vkuzo commented Feb 26, 2025

Summary:

Various changes to beef up float8 and mx roofline estimations.

Test Plan:

https://gist.github.com/vkuzo/786f4d3aef2a042ca4d8ef40dbd35d64
https://gist.github.com/vkuzo/75c46bb4b0f5bf0672223230c8dda78a


python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --float8_recipe_name rowwise
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --mx_recipe_name mxfp8_emulated

Reviewers:

Subscribers:

Tasks:

Tags:

vkuzo added 13 commits February 25, 2025 14:03
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
@vkuzo
Copy link
Contributor Author

vkuzo commented Feb 26, 2025

@pytorch-bot
Copy link

pytorch-bot bot commented Feb 26, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/ao/1789

Note: Links to docs will display an error until the docs builds have been completed.

❌ 2 New Failures

As of commit 9bedb01 with merge base b9c51b7 (image):

NEW FAILURES - The following jobs have failed:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 26, 2025
vkuzo added a commit that referenced this pull request Feb 26, 2025
Summary:

Test Plan:

```
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --float8_recipe_name rowwise
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --mx_recipe_name mxfp8_emulated
```

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 01f4cf1
ghstack-comment-id: 2686473047
Pull Request resolved: #1789
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
vkuzo added a commit that referenced this pull request Feb 27, 2025
Summary:

Test Plan:

```
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --float8_recipe_name rowwise
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --mx_recipe_name mxfp8_emulated
```

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 01f4cf1
ghstack-comment-id: 2686473047
Pull Request resolved: #1789
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
vkuzo added a commit that referenced this pull request Feb 27, 2025
Summary:

Test Plan:

```
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --float8_recipe_name rowwise
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --mx_recipe_name mxfp8_emulated
```

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 01f4cf1
ghstack-comment-id: 2686473047
Pull Request resolved: #1789
[ghstack-poisoned]
[ghstack-poisoned]
vkuzo added a commit that referenced this pull request Feb 27, 2025
Summary:

Test Plan:

```
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --float8_recipe_name rowwise
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --mx_recipe_name mxfp8_emulated
```

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 01f4cf1
ghstack-comment-id: 2686473047
Pull Request resolved: #1789
[ghstack-poisoned]
@vkuzo vkuzo changed the base branch from gh/vkuzo/45/head to main February 27, 2025 16:15
vkuzo added a commit that referenced this pull request Feb 27, 2025
Summary:

Test Plan:

```
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --float8_recipe_name rowwise
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --mx_recipe_name mxfp8_emulated
```

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 01f4cf1
ghstack-comment-id: 2686473047
Pull Request resolved: #1789
[ghstack-poisoned]
vkuzo added a commit that referenced this pull request Feb 28, 2025
Summary:

Test Plan:

```
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --float8_recipe_name rowwise
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --mx_recipe_name mxfp8_emulated
```

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 9b83266
ghstack-comment-id: 2686473047
Pull Request resolved: #1789
[ghstack-poisoned]
vkuzo added a commit that referenced this pull request Feb 28, 2025
Summary:

Test Plan:

```
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --float8_recipe_name rowwise
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --mx_recipe_name mxfp8_emulated
```

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 19c2e7c
ghstack-comment-id: 2686473047
Pull Request resolved: #1789
[ghstack-poisoned]
vkuzo added a commit that referenced this pull request Feb 28, 2025
Summary:

Test Plan:

```
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --float8_recipe_name rowwise
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --mx_recipe_name mxfp8_emulated
```

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: a34af93
ghstack-comment-id: 2686473047
Pull Request resolved: #1789
[ghstack-poisoned]
vkuzo added a commit that referenced this pull request Feb 28, 2025
Summary:

Test Plan:

```
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --float8_recipe_name rowwise
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --mx_recipe_name mxfp8_emulated
```

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: fbc134f
ghstack-comment-id: 2686473047
Pull Request resolved: #1789
@vkuzo vkuzo added the topic: not user facing Use this tag if you don't want this PR to show up in release notes label Feb 28, 2025
[ghstack-poisoned]
vkuzo added a commit that referenced this pull request Mar 1, 2025
Summary:

Test Plan:

```
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --float8_recipe_name rowwise
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --mx_recipe_name mxfp8_emulated
```

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 7857b5c
ghstack-comment-id: 2686473047
Pull Request resolved: #1789
vkuzo added a commit that referenced this pull request Mar 1, 2025
Summary:

Test Plan:

```
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --float8_recipe_name rowwise
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --mx_recipe_name mxfp8_emulated
```

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 7857b5c
ghstack-comment-id: 2686473047
Pull Request resolved: #1789
[ghstack-poisoned]
vkuzo added a commit that referenced this pull request Mar 3, 2025
Summary:

Test Plan:

```
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --float8_recipe_name rowwise
python benchmarks/float8/float8_roofline.py ~/local/tmp/20250226_test.csv --n_limit 1 --mx_recipe_name mxfp8_emulated
```

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: a85a625
ghstack-comment-id: 2686473047
Pull Request resolved: #1789
@vkuzo vkuzo merged commit d8af7d7 into main Mar 4, 2025
44 of 47 checks passed
Jack-Khuu pushed a commit that referenced this pull request Mar 4, 2025
* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]
liangel-02 pushed a commit that referenced this pull request Aug 25, 2025
* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Update

[ghstack-poisoned]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. topic: not user facing Use this tag if you don't want this PR to show up in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants