Skip to content

Conversation

fegin
Copy link
Contributor

@fegin fegin commented Sep 23, 2025

Allows to configure flight recorder file prefix

Allows to configure flight recorder file prefix
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Sep 23, 2025

TRACE_BUFFER_SIZE = "TORCH_FR_BUFFER_SIZE"
TRACE_FILE = "TORCH_NCCL_DEBUG_INFO_TEMP_FILE"
TRACE_FILE = "TORCH_FR_DUMP_TEMP_FILE"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

TORCH_NCCL_DEBUG_INFO_TEMP_FILE is being deprecated

Copy link
Contributor

Choose a reason for hiding this comment

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

I do think we should work on #1192

Copy link
Contributor

@wwwjn wwwjn 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 pytorch deleted a comment from pytorch-bot bot Sep 23, 2025
@fegin fegin merged commit 0f34257 into main Sep 23, 2025
7 of 8 checks passed

TRACE_BUFFER_SIZE = "TORCH_FR_BUFFER_SIZE"
TRACE_FILE = "TORCH_NCCL_DEBUG_INFO_TEMP_FILE"
TRACE_FILE = "TORCH_FR_DUMP_TEMP_FILE"
Copy link
Contributor

Choose a reason for hiding this comment

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

I do think we should work on #1192

@tianyu-l tianyu-l deleted the chienchin/fr_pr branch September 23, 2025 20:49
@fegin
Copy link
Contributor Author

fegin commented Sep 23, 2025

I do think we should work on #1192

Good point. We can address this issue after PTC.

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