Skip to content

Conversation

danielvegamyhre
Copy link
Contributor

Torchtitan logs now output an extra space between some sections, which breaks the regex in this log parsing script.

Updated regex to be more flexible to fix and avoid this happening again.

Copy link

pytorch-bot bot commented Oct 1, 2025

🔗 Helpful Links

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

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

⏳ No Failures, 2 Pending

As of commit 2c62746 with merge base cbd3adb (image):
💚 Looks good so far! There are no failures yet. 💚

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

@meta-cla meta-cla 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 Oct 1, 2025
@danielvegamyhre danielvegamyhre added topic: bug fix Use this tag for PRs that fix bugs moe and removed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels Oct 1, 2025
@danielvegamyhre danielvegamyhre merged commit 28f45e3 into main Oct 1, 2025
20 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

moe topic: bug fix Use this tag for PRs that fix bugs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants