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.

Sign up for GitHub

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

Jump to bottom

Enable perf reporting for reduce-scatter async, all-gather async, all-reduce async (when added) #16648

Open
6 tasks
Tracked by #16640
SeanNijjar opened this issue Jan 11, 2025 · 0 comments
Open
6 tasks
Tracked by #16640

Enable perf reporting for reduce-scatter async, all-gather async, all-reduce async (when added) #16648

SeanNijjar opened this issue Jan 11, 2025 · 0 comments
Assignees
@Aswinmcw
Labels
op_cat: ccl

Comments

@SeanNijjar
Copy link
Contributor

SeanNijjar commented Jan 11, 2025 •
edited
Loading

  • Perf reporting for all-gather async
    • T3K tests here for reference:: tests/ttnn/unit_tests/operations/ccl/test_new_all_gather.py
    • TG functional tests: coordinate with @caixunshiren who is adding these
  • Perf reporting for reduce-scatter async:
    • T3K tests for reference: tests/ttnn/unit_tests/operations/ccl/test_reduce_scatter_async.py
    • TG functional tests: coordinate with @caixunshiren who is adding these
@SeanNijjar SeanNijjar mentioned this issue Jan 11, 2025
Open
13 tasks
This was referenced Jan 24, 2025
Merged
Merged
Aswinmcw added a commit that referenced this issue Jan 30, 2025
@Aswinmcw
Add perf reporting for ccl async mode (#16658)
32bb829 
### Ticket
#16648 

### Problem description
Need perf reporting for async all gather

### What's changed
<img width="1292" alt="Screenshot 2025-01-24 at 8 42 29 PM"
src="https://github.com/user-attachments/assets/33c64cf1-9f31-4567-89ed-813acc16e49d"
/>


### Checklist
- [ ] Post commit CI passes
- [ ] Blackhole Post commit (if applicable)
- [ ] Model regression CI testing passes (if applicable)
- [ ] Device performance regression CI testing passes (if applicable)
- [ ] **(For models and ops writers)** Full [new
models](https://github.com/tenstorrent/tt-metal/actions/workflows/full-new-models-suite.yaml)
tests passes
- [ ] New/Existing tests provide coverage for changes
Aswinmcw added a commit that referenced this issue Jan 30, 2025
@Aswinmcw
Add perf report for reduce scatter async (#17223)
4274236 
### Ticket
#16648 

### Problem description
Provide context for the problem.

### What's changed

<img width="1299" alt="Screenshot 2025-01-28 at 5 41 07 PM"
src="https://github.com/user-attachments/assets/4c9353a0-d001-470d-8417-56cccb661ddd"
/>


### Checklist
- [ ] Post commit CI passes
- [ ] Blackhole Post commit (if applicable)
- [ ] Model regression CI testing passes (if applicable)
- [ ] Device performance regression CI testing passes (if applicable)
- [ ] **(For models and ops writers)** Full [new
models](https://github.com/tenstorrent/tt-metal/actions/workflows/full-new-models-suite.yaml)
tests passes
- [ ] New/Existing tests provide coverage for changes
williamlyTT pushed a commit that referenced this issue Jan 30, 2025
@Aswinmcw @williamlyTT
Add perf reporting for ccl async mode (#16658)
826e408 
### Ticket
#16648 

### Problem description
Need perf reporting for async all gather

### What's changed
<img width="1292" alt="Screenshot 2025-01-24 at 8 42 29 PM"
src="https://github.com/user-attachments/assets/33c64cf1-9f31-4567-89ed-813acc16e49d"
/>


### Checklist
- [ ] Post commit CI passes
- [ ] Blackhole Post commit (if applicable)
- [ ] Model regression CI testing passes (if applicable)
- [ ] Device performance regression CI testing passes (if applicable)
- [ ] **(For models and ops writers)** Full [new
models](https://github.com/tenstorrent/tt-metal/actions/workflows/full-new-models-suite.yaml)
tests passes
- [ ] New/Existing tests provide coverage for changes
williamlyTT pushed a commit that referenced this issue Jan 30, 2025
@Aswinmcw @williamlyTT
Add perf report for reduce scatter async (#17223)
51a296e 
### Ticket
#16648 

### Problem description
Provide context for the problem.

### What's changed

<img width="1299" alt="Screenshot 2025-01-28 at 5 41 07 PM"
src="https://github.com/user-attachments/assets/4c9353a0-d001-470d-8417-56cccb661ddd"
/>


### Checklist
- [ ] Post commit CI passes
- [ ] Blackhole Post commit (if applicable)
- [ ] Model regression CI testing passes (if applicable)
- [ ] Device performance regression CI testing passes (if applicable)
- [ ] **(For models and ops writers)** Full [new
models](https://github.com/tenstorrent/tt-metal/actions/workflows/full-new-models-suite.yaml)
tests passes
- [ ] New/Existing tests provide coverage for changes
yieldthought pushed a commit that referenced this issue Jan 31, 2025
@Aswinmcw @yieldthought
Add perf reporting for ccl async mode (#16658)
08b7e9e 
### Ticket
#16648 

### Problem description
Need perf reporting for async all gather

### What's changed
<img width="1292" alt="Screenshot 2025-01-24 at 8 42 29 PM"
src="https://github.com/user-attachments/assets/33c64cf1-9f31-4567-89ed-813acc16e49d"
/>


### Checklist
- [ ] Post commit CI passes
- [ ] Blackhole Post commit (if applicable)
- [ ] Model regression CI testing passes (if applicable)
- [ ] Device performance regression CI testing passes (if applicable)
- [ ] **(For models and ops writers)** Full [new
models](https://github.com/tenstorrent/tt-metal/actions/workflows/full-new-models-suite.yaml)
tests passes
- [ ] New/Existing tests provide coverage for changes
yieldthought pushed a commit that referenced this issue Jan 31, 2025
@Aswinmcw @yieldthought
Add perf report for reduce scatter async (#17223)
b3213e4 
### Ticket
#16648 

### Problem description
Provide context for the problem.

### What's changed

<img width="1299" alt="Screenshot 2025-01-28 at 5 41 07 PM"
src="https://github.com/user-attachments/assets/4c9353a0-d001-470d-8417-56cccb661ddd"
/>


### Checklist
- [ ] Post commit CI passes
- [ ] Blackhole Post commit (if applicable)
- [ ] Model regression CI testing passes (if applicable)
- [ ] Device performance regression CI testing passes (if applicable)
- [ ] **(For models and ops writers)** Full [new
models](https://github.com/tenstorrent/tt-metal/actions/workflows/full-new-models-suite.yaml)
tests passes
- [ ] New/Existing tests provide coverage for changes
nikileshx pushed a commit to nikileshx/tt-metal that referenced this issue Feb 3, 2025
@Aswinmcw @nikileshx
Add perf reporting for ccl async mode (tenstorrent#16658)
fe09b77 
### Ticket
tenstorrent#16648 

### Problem description
Need perf reporting for async all gather

### What's changed
<img width="1292" alt="Screenshot 2025-01-24 at 8 42 29 PM"
src="https://github.com/user-attachments/assets/33c64cf1-9f31-4567-89ed-813acc16e49d"
/>


### Checklist
- [ ] Post commit CI passes
- [ ] Blackhole Post commit (if applicable)
- [ ] Model regression CI testing passes (if applicable)
- [ ] Device performance regression CI testing passes (if applicable)
- [ ] **(For models and ops writers)** Full [new
models](https://github.com/tenstorrent/tt-metal/actions/workflows/full-new-models-suite.yaml)
tests passes
- [ ] New/Existing tests provide coverage for changes
nikileshx pushed a commit to nikileshx/tt-metal that referenced this issue Feb 3, 2025
@Aswinmcw @nikileshx
Add perf report for reduce scatter async (tenstorrent#17223)
87933e5 
### Ticket
tenstorrent#16648 

### Problem description
Provide context for the problem.

### What's changed

<img width="1299" alt="Screenshot 2025-01-28 at 5 41 07 PM"
src="https://github.com/user-attachments/assets/4c9353a0-d001-470d-8417-56cccb661ddd"
/>


### Checklist
- [ ] Post commit CI passes
- [ ] Blackhole Post commit (if applicable)
- [ ] Model regression CI testing passes (if applicable)
- [ ] Device performance regression CI testing passes (if applicable)
- [ ] **(For models and ops writers)** Full [new
models](https://github.com/tenstorrent/tt-metal/actions/workflows/full-new-models-suite.yaml)
tests passes
- [ ] New/Existing tests provide coverage for changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Aswinmcw Aswinmcw

Labels
op_cat: ccl
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@SeanNijjar @Aswinmcw