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

Update dpugen to 0.2.3 and saic to latest master #635

Merged
merged 3 commits into from
Oct 30, 2024

Conversation

vikumarks
Copy link
Contributor

Update SAI-Challenger @ f249b33 to latest master
dpugen 0.0.5 to 0.2.3

vikumarks and others added 3 commits October 29, 2024 10:21
update from dpugen==0.0.5 to dpugen==0.2.3
…ger-bldr

 dpugen version update and update to latest saichallenge
@r12f
Copy link
Collaborator

r12f commented Oct 30, 2024

hi Vinod, do you mind to update the PR title to something like "Update dpugen to 0.2.3 and saic to latest master"?

@vikumarks vikumarks changed the title Pr update dpugen saic Update dpugen to 0.2.3 and saic to latest master Oct 30, 2024
@vikumarks
Copy link
Contributor Author

hi Vinod, do you mind to update the PR title to something like "Update dpugen to 0.2.3 and saic to latest master"?

Updated title as requested.

Copy link
Collaborator

@r12f r12f left a comment

Choose a reason for hiding this comment

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

lgtm!

@r12f r12f merged commit 0b8d12c into sonic-net:main Oct 30, 2024
3 checks passed
@r12f
Copy link
Collaborator

r12f commented Oct 30, 2024

thanks Vinod! it is merged now.

vikumarks added a commit to vikumarks/DASH that referenced this pull request Oct 30, 2024
Issue sonic-net#581
fixed by PR  sonic-net#635
So enabling back the sai-c tests.
vikumarks added a commit to vikumarks/DASH that referenced this pull request Oct 31, 2024
enabled saichallenger-functional and scale as dpugen issue is fixed by sonic-net#635
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.

2 participants