Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[debug][dtensor] hosting commdebugmode browser's visualizer PyTorch #1699

Open
wants to merge 3 commits into
base: site
Choose a base branch
from

Conversation

sinhaanshul
Copy link

One of CommDebugMode's features is that it allows users to create a json file using the debugger's output and create a tree structure of the results in a browser visualizer. I would like to host this html file on pytorch so that users don't have to constantly open the html file manually every single time they want to use it.

Copy link

netlify bot commented Aug 1, 2024

👷 Deploy Preview for pytorch-dot-org-preview processing.

Name Link
🔨 Latest commit 7ad1aab
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-dot-org-preview/deploys/66ac023fef8ab6000865f79e

</div>
<div id="tree-container"></div>

<script src="https://cdn.jsdelivr.net/gh/pytorch/pytorch@main/torch/distributed/_tensor/debug/comm_mode_broswer_visual.js"></script>
Copy link
Contributor

Choose a reason for hiding this comment

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

What's wrong with referencing github script directly?

Suggested change
<script src="https://cdn.jsdelivr.net/gh/pytorch/pytorch@main/torch/distributed/_tensor/debug/comm_mode_broswer_visual.js"></script>
<script src="https://raw.githubusercontent.com/pytorch/pytorch/main/torch/distributed/_tensor/debug/comm_mode_broswer_visual.js"></script>

@wanchaol
Copy link
Contributor

wanchaol commented Aug 2, 2024

Maybe we can start with a private html (with _ prefix) for us to test on various use cases and then move it to public?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants