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

Information redundancy #42

Closed
XcodeRole opened this issue Apr 27, 2024 · 4 comments
Closed

Information redundancy #42

XcodeRole opened this issue Apr 27, 2024 · 4 comments
Labels
enhancement New feature or request help wanted Extra attention is needed question Further information is requested

Comments

@XcodeRole
Copy link

There is too much information from pytorch ET plus, including function call relationships with too many details, but in fact we only focus on computing, memory access and communication.
Does such a large amount of information run counter to the design philosophy of chakra?

@XcodeRole XcodeRole added the question Further information is requested label Apr 27, 2024
@TaekyungHeo
Copy link
Contributor

Thank you for raising a concern, @XcodeRole. Could you please specify which information appears redundant? The Chakra working group welcomes any suggestions for improved design and engineering. If you are referring to the function call relationships, this information is crucial for one of our key downstream use cases, namely simulation. We must maintain the function call relationships to replay operators in a simulator.

@XcodeRole
Copy link
Author

XcodeRole commented May 9, 2024

In Chakra ET, nodes are represented on a per-function-call basis. Many functions have an execution time of 0 microseconds because they do not perform computations themselves; instead, they invoke computational functions. However, these still occupy a node in ET, leading to a very large and redundant call graph.

@TaekyungHeo
Copy link
Contributor

Yes, it makes sense if that is the case. Do you have any example traces that exhibit such behaviors? I had an opportunity to collect LLAMA traces, but I did not observe any 0-microsecond operators. This might be due to rounding, but I believe the measurements are accurate because I have validated them, as shown in the following PR.

@XcodeRole
Copy link
Author

XcodeRole commented May 9, 2024

You responded very quickly, which was a pleasant surprise. However, the debug code you recently merged is inconsistent with the version I am using. There were some bugs in the version I was working with, and I had made my own modifications to fix them. Therefore, it will take me some time to rerun tests to ensure the changes are compatible with the updated code.

@srinivas212 srinivas212 added enhancement New feature or request help wanted Extra attention is needed labels May 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants