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

Xiyon/add cb for targetqps #1120

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

xy223
Copy link

@xy223 xy223 commented Mar 10, 2022

No description provided.

@github-actions
Copy link

MLCommons CLA bot:
Thank you for your submission, we really appreciate it. We ask that you all sign our MLCommons CLA and be a member before we can accept your contribution. If you are interested in membership, please contact [email protected] .
0 out of 2 committers have signed the MLCommons CLA.
@Xin Yong
@xy223
Xin Yong seems not to be a GitHub user. You need a GitHub account after you become MLCommons member. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request

@psyhtest
Copy link
Contributor

@xy223 Can you please provide some rationale for this change? Also, is your organization a member of MLCommons? Many thanks.

@lgtm-com
Copy link

lgtm-com bot commented Apr 4, 2022

This pull request introduces 1 alert when merging e0feefd into aaa8f88 - view on LGTM.com

new alerts:

  • 1 for Missing return statement

@mrmhodak
Copy link
Contributor

mrmhodak commented Nov 7, 2023

@pgmpablo157321 : Can you have a look?

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.

4 participants