Skip to content

Conversation

YUNQIUGUO
Copy link
Contributor

@YUNQIUGUO YUNQIUGUO commented Feb 12, 2025

Summary:
per request, explicitly dumping the float dtypes for aten tensors in debug printing summary info.

can be useful in identifying issues such as "wrong AOTI Lowering precisions"

Test Plan:

 AOT_INDUCTOR_DEBUG_INTERMEDIATE_VALUE_PRINTER=2 TORCH_LOGS="+inductor, output_code" buck2 run -c fbcode.enable_gpu_sections=true -c fbcode.nvcc_arch=h100 @//mode/opt fbcode//caffe2/test/inductor:test_aot_inductor -- -r test_addmm

Differential Revision: D69547344

cc @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @ipiszy @yf225 @chenyang78 @kadeng @muchulee8 @amjames @chauhang @aakhundov

Copy link

pytorch-bot bot commented Feb 12, 2025

🔗 Helpful Links

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

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

⏳ 5 Pending, 4 Unrelated Failures

As of commit 53c9e00 with merge base 67cbbb2 (image):

BROKEN TRUNK - The following jobs failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

UNSTABLE - The following jobs are marked as unstable, possibly due to flakiness on trunk:

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

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

@YUNQIUGUO YUNQIUGUO added the topic: not user facing topic category label Feb 12, 2025
Copy link
Contributor

@jingsh jingsh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Feb 12, 2025
Copy link
Contributor

@ColinPeppler ColinPeppler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to add prints for integer types too

@facebook-github-bot
Copy link
Contributor

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

…6 data type (pytorch#147020)

Summary:

Per request, explicitly dumping the float dtypes for aten tensors in debug printing summary info.

can be useful in identifying issues associated with "wrong AOTI lowering input precisions"

before this diff:
[  before_launch - triton_tem_fused_addmm_1 - buf1  ]:
Number of elements: 6144
**Dtype: Float**
Mean value: 0.0195527
Min value: 0
Max value: 4.55529
Device: cuda:0
Size: [1024, 6]
Stride: [6, 1]
Layout: Strided
Is contiguous: 1
Requires grad: 0


after this diff:
[  before_launch - triton_tem_fused_addmm_1 - buf1  ]:
Number of elements: 6144
**Dtype: float32**
Mean value: 0.0195527
Min value: 0
Max value: 4.55529
Device: cuda:0
Size: [1024, 6]
Stride: [6, 1]
Layout: Strided
Is contiguous: 1
Requires grad: 0

Test Plan:
```
 AOT_INDUCTOR_DEBUG_INTERMEDIATE_VALUE_PRINTER=2 TORCH_LOGS="+inductor, output_code" buck2 run -c fbcode.enable_gpu_sections=true -c fbcode.nvcc_arch=h100 @//mode/opt fbcode//caffe2/test/inductor:test_aot_inductor -- -r test_addmm
```

Reviewed By: hl475, jingsh, ColinPeppler

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

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

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge

(Initiating merge automatically since Phabricator Diff has merged)

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

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.

5 participants