Skip to content

Conversation

@yf225
Copy link
Contributor

@yf225 yf225 commented Oct 24, 2025

@yf225 yf225 requested review from jansel and oulgen October 24, 2025 20:02
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Oct 24, 2025
@yf225 yf225 force-pushed the ci_amd_journal_errors branch from b71644e to a2905be Compare October 24, 2025 20:04
@yf225 yf225 changed the title [CI] Fix AMD expected journal errors [CI] Work around AMD expected journal errors Oct 24, 2025
@yf225 yf225 force-pushed the ci_amd_journal_errors branch from a2905be to ad79780 Compare October 24, 2025 20:04
@yf225 yf225 changed the title [CI] Work around AMD expected journal errors [CI] Skip expected journal checks for specific tests on AMD Oct 24, 2025
@yf225 yf225 force-pushed the ci_amd_journal_errors branch from ad79780 to b7e6cb8 Compare October 24, 2025 20:05
@yf225 yf225 changed the title [CI] Skip expected journal checks for specific tests on AMD [CI] Skip expected journal check for specific tests on AMD Oct 24, 2025
Copy link
Contributor

@oulgen oulgen left a comment

Choose a reason for hiding this comment

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

You should instead do exactly this

8ff3234

@yf225 yf225 force-pushed the ci_amd_journal_errors branch from b7e6cb8 to 4cf2672 Compare October 24, 2025 20:18
@yf225 yf225 changed the title [CI] Skip expected journal check for specific tests on AMD [CI] Fix AMD journal check errors Oct 24, 2025
Comment on lines +776 to +777
# libdevice.rsqrt( -> tl.rsqrt(
code = re.sub(r"\blibdevice\.rsqrt\s*\(", "tl.rsqrt(", code)
Copy link
Contributor

Choose a reason for hiding this comment

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

bad pasta?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this one actually shows up in AMD tests:

2025-10-24T16:25:05.3039413Z E   -     v_8 = tl.rsqrt(v_7)
2025-10-24T16:25:05.3039489Z E   ?           -
2025-10-24T16:25:05.3039637Z E   +     v_8 = libdevice.rsqrt(v_7)
2025-10-24T16:25:05.3039708Z E   ?            ++++++++

@yf225 yf225 force-pushed the ci_amd_journal_errors branch 4 times, most recently from 7ac0c8d to 0fdd940 Compare October 24, 2025 23:32
@yf225 yf225 force-pushed the ci_amd_journal_errors branch from 0fdd940 to a75a29b Compare October 24, 2025 23:38
@yf225 yf225 merged commit 38675f9 into main Oct 25, 2025
14 checks passed
tianrengao pushed a commit that referenced this pull request Nov 5, 2025
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 Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants