Skip to content

[FIX] Updated readme.md and tracer.py - #117

Closed
gujialiang123 wants to merge 12 commits into
Deep-Learning-Profiling-Tools:mainfrom
gujialiang123:main
Closed

gujialiang123 wants to merge 12 commits into
Deep-Learning-Profiling-Tools:mainfrom
gujialiang123:main

Conversation

@gujialiang123

Copy link
Copy Markdown
Collaborator

1 Fixed a problem in tracer.py where the register_for_loop_callback function was missing a None return value.
2 Updated the installation process in readme. You can directly install the CPU version of Torch, so there is no need to uninstall pytorch-triton. the previous installation steps do not work on my computer.

@mark14wu

Copy link
Copy Markdown
Collaborator

#106 Fixed in this PR.

@mark14wu mark14wu changed the title Updated readme.md and Tracer.py [FIX] Updated readme.md and tracer.py Aug 28, 2025
@mark14wu mark14wu changed the title [FIX] Updated readme.md and tracer.py [FIX] Updated readme.md and tracer.py Aug 28, 2025
Comment thread README.md

```sh
pip install --pre torch torchvision torchaudio --index-url https://download.pytorch.org/whl/nightly/cu121
pip install --pre torch torchvision torchaudio --index-url https://download.pytorch.org/whl/cpu

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is not 100% correct. We don't rely on either torch-cpu or torch-gpu

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Better to just refer people to the pytorch installation page

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

And say it's up to them to install any torch version they like. We support it anyway.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@gujialiang123 This hasn't been addressed yet

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Done! Please check it!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Oh, what I meant is much simpler.

Just refer to this page https://pytorch.org/get-started/locally/

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Without putting any concrete instructions

@mark14wu

Copy link
Copy Markdown
Collaborator

@Jokeren Hi Keren, could you add Jialiang as a maintainer of triton-viz?
@gujialiang123 Hi Jialiang, please run pre-commit run --all-files to fix format problems.

@Jokeren

Jokeren commented Aug 28, 2025

Copy link
Copy Markdown
Member

@gujialiang123 I sent you two invitations. Please accept

@gujialiang123

Copy link
Copy Markdown
Collaborator Author

@gujialiang123 I sent you two invitations. Please accept

@Jokeren Thanks! I have accepted them!

@gujialiang123

Copy link
Copy Markdown
Collaborator Author

@Jokeren Hi Keren, could you add Jialiang as a maintainer of triton-viz? @gujialiang123 Hi Jialiang, please run pre-commit run --all-files to fix format problems.

@mark14wu Done!

Comment thread README.md

```sh
pip install --pre torch torchvision torchaudio --index-url https://download.pytorch.org/whl/nightly/cu121
pip install --pre torch torchvision torchaudio --index-url https://download.pytorch.org/whl/cpu

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@gujialiang123 This hasn't been addressed yet



def collect_grid():
# 动态获取当前的 launches,尽最大可能拿到"非空"的那份

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No comments in Chinese...

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@gujialiang123 also you don't have to upload any ipynb files

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@gujialiang123 also you don't have to upload any ipynb files

Sorry, I got the wrong branch... I'll fix these problems later, sorry.

@gujialiang123
gujialiang123 marked this pull request as draft September 18, 2025 18:18
@mark14wu

Copy link
Copy Markdown
Collaborator

Abandoned.

@mark14wu mark14wu closed this Sep 29, 2025
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