Add Cadence Computation Profiling Guide #1636
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
turbolent
left a comment
turbolent
left a comment
There was a problem hiding this comment.
Thank you for writing this up! Some small comments, but looks great overall, the step-by-steps instructions and troubleshooting info is very useful 👍
|
|
||
| ## Using Source File Pragmas | ||
|
|
||
| The `#sourceFile` pragma improves report readability by associating your code with meaningful file paths. Without it, reports show generic identifiers. |
There was a problem hiding this comment.
I'm not sure if sourceFile pragma is supported when profiling. Maybe it only applies to computation reports? I didn't add explicit support for source file pragmas when implementing profiling, but maybe this already works automatically somehow?
Maybe double check availability, and point out if it only applies to reporting. If it's not supported for profiling yet, let's add it, it looks like a nice feature!
There was a problem hiding this comment.
It looks like it only applies to reporting. Good catch. I'll adjust the doc.
There was a problem hiding this comment.
When it gets added we can update the doc.
Co-authored-by: Bastian Müller <bastian@turbolent.com>
No description provided.