Skip to content

Add PYTORCH_TEST_WITH_ROCM=1 to test_profiler test for CI (#1289)#1289

Closed
ryanzhang22 wants to merge 1 commit intopytorch:mainfrom
ryanzhang22:export-D95839791
Closed

Add PYTORCH_TEST_WITH_ROCM=1 to test_profiler test for CI (#1289)#1289
ryanzhang22 wants to merge 1 commit intopytorch:mainfrom
ryanzhang22:export-D95839791

Conversation

@ryanzhang22
Copy link
Copy Markdown
Contributor

@ryanzhang22 ryanzhang22 commented Mar 9, 2026

Summary:

For #1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791

@pytorch-bot
Copy link
Copy Markdown

pytorch-bot bot commented Mar 9, 2026

Warning: Unknown label ciflow/rocm-mi300.
Currently recognized labels are

  • ciflow/rocm

Please add the new label to .github/pytorch-probot.yml

@meta-codesync
Copy link
Copy Markdown

meta-codesync bot commented Mar 9, 2026

@ryanzhang22 has exported this pull request. If you are a Meta employee, you can view the originating Diff in D95839791.

@meta-codesync meta-codesync bot changed the title Add PYTORCH_TEST_WITH_ROCM=1 to test_profiler test for CI Add PYTORCH_TEST_WITH_ROCM=1 to test_profiler test for CI (#1289) Mar 17, 2026
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 17, 2026
Summary:

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 17, 2026
Summary:

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 17, 2026
Summary:
Pull Request resolved: pytorch#1289

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 17, 2026
Summary:

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 17, 2026
Summary:
Pull Request resolved: pytorch#1289

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
@ryanzhang22 ryanzhang22 force-pushed the export-D95839791 branch 2 times, most recently from dbeccaa to 7e5eea0 Compare March 18, 2026 15:30
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 18, 2026
Summary:

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 18, 2026
Summary:

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 18, 2026
Summary:
Pull Request resolved: pytorch#1289

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 18, 2026
Summary:
Pull Request resolved: pytorch#1289

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
@ryanzhang22 ryanzhang22 force-pushed the export-D95839791 branch 2 times, most recently from 30afc13 to 32973cf Compare March 19, 2026 20:52
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 19, 2026
Summary:

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 19, 2026
Summary:
Pull Request resolved: pytorch#1289

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 19, 2026
Summary:

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 19, 2026
Summary:
Pull Request resolved: pytorch#1289

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 20, 2026
Summary:

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 20, 2026
Summary:
Pull Request resolved: pytorch#1289

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 20, 2026
Summary:

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 20, 2026
Summary:
Pull Request resolved: pytorch#1289

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 23, 2026
Summary:

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
@ryanzhang22 ryanzhang22 force-pushed the export-D95839791 branch 2 times, most recently from 2d9230d to 22a1c82 Compare March 23, 2026 12:54
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 23, 2026
Summary:

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
ryanzhang22 added a commit to ryanzhang22/kineto that referenced this pull request Mar 23, 2026
Summary:
Pull Request resolved: pytorch#1289

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
Summary:
Pull Request resolved: pytorch#1289

For pytorch#1242

I believe this is causing us to not properly set TEST_WITH_ROCM in the ROCm CI tests, so let's set the flag then un-deselect the test.

Reviewed By: scotts

Differential Revision: D95839791
@meta-codesync
Copy link
Copy Markdown

meta-codesync bot commented Mar 23, 2026

This pull request has been merged in 8b42d4c.

smalltalkman pushed a commit to smalltalkman/pytorch that referenced this pull request Mar 24, 2026
Contains the following changes:

- [pytorch#1289](pytorch/kineto#1289) - Add `PYTORCH_TEST_WITH_ROCM=1` to profiler CI tests
- [pytorch#1317](pytorch/kineto#1317) - Move activity_type enum/string map to header file
- [pytorch#1315](pytorch/kineto#1315) - Update .gitignore
- [pytorch#1320](pytorch/kineto#1320) - Disable `test_schedule_function_count` (segfault fix)
- [pytorch#1312](pytorch/kineto#1312) - Add missing `<chrono>` header for Windows builds
- [pytorch#1310](pytorch/kineto#1310) - Start flows on `hipGraphLaunch`
Pull Request resolved: pytorch#178286
Approved by: https://github.com/scotts, https://github.com/Skylion007
Copilot AI pushed a commit to pytorch/pytorch that referenced this pull request Mar 27, 2026
Contains the following changes:

- [#1289](pytorch/kineto#1289) - Add `PYTORCH_TEST_WITH_ROCM=1` to profiler CI tests
- [#1317](pytorch/kineto#1317) - Move activity_type enum/string map to header file
- [#1315](pytorch/kineto#1315) - Update .gitignore
- [#1320](pytorch/kineto#1320) - Disable `test_schedule_function_count` (segfault fix)
- [#1312](pytorch/kineto#1312) - Add missing `<chrono>` header for Windows builds
- [#1310](pytorch/kineto#1310) - Start flows on `hipGraphLaunch`
Pull Request resolved: #178286
Approved by: https://github.com/scotts, https://github.com/Skylion007

Co-authored-by: Xia-Weiwen <12522207+Xia-Weiwen@users.noreply.github.com>
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.

2 participants