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.

Sign up for GitHub

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

Jump to bottom

Update benchmark carrying capacity #274

Merged
merged 2 commits into from
Oct 17, 2023
Merged

Update benchmark carrying capacity #274

merged 2 commits into from
Oct 17, 2023

Conversation

mrasquinha-g
Copy link
Contributor

Update rules to indicate carrying capacity.

@mrasquinha-g
Update inference_rules.adoc
11fd721 
Update rules to indicate carrying capacity.
@mrasquinha-g mrasquinha-g requested a review from a team as a code owner June 20, 2023 02:16
@github-actions
Copy link

github-actions bot commented Jun 20, 2023 •
edited
Loading

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

@mrasquinha-g mrasquinha-g changed the title Update inference_rules.adoc Update benchmark carrying capacity Jun 20, 2023
@psyhtest
Copy link
Contributor

Each suite has a carrying capacity of 10 benchmarks
i.e at any point in time, the number of benchmarks will not exceed 10.

Theoretically, the Datacenter and Edge benchmarks could be completely disjoint, so the number of benchmarks would be 20.

@mrmhodak
Copy link
Contributor

mrmhodak commented Jul 5, 2023

Needs to be clarified

mrmhodak
mrmhodak approved these changes Oct 17, 2023
View reviewed changes
Copy link
Contributor

@mrmhodak mrmhodak left a comment

Choose a reason for hiding this comment

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

Approved after WG discussion

@mrmhodak mrmhodak merged commit d850817 into master Oct 17, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Oct 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mrmhodak mrmhodak mrmhodak approved these changes

@psyhtest psyhtest psyhtest approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mrasquinha-g @psyhtest @mrmhodak