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

CR-1227470 Fix for empty view in waveform for few interface tile metric sets for VE2 #8731

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

parthash0804
Copy link
Collaborator

@parthash0804 parthash0804 commented Jan 30, 2025

Problem solved by the commit

Incorrect information in aie_event_runtime_config.json was causing the waveform to be empty.

Bug / issue (if any) fixed, which PR introduced the bug, how it was discovered

#8724

How problem was solved, alternative solutions (if any) and why they were rejected

Added the new events to the functions which return channel number so that correct information can be added in aie_event_runtime_config.json file.

Risks (if any) associated the changes in the commit

Low

What has been tested and how, request additional testing if necessary

Tested compilation of VE2 and generated trace for a design and examined the waveform.
Tested client compilation.

Documentation impact (if any)

NA

@parthash0804 parthash0804 requested a review from pgschuey January 30, 2025 16:53
@parthash0804 parthash0804 requested review from jvillarre and removed request for jvillarre January 30, 2025 16:53
@jvillarre jvillarre merged commit 02c4811 into Xilinx:master Jan 30, 2025
20 checks passed
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