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

Fix regex #1178

Merged
merged 1 commit into from
Jul 18, 2022
Merged

Fix regex #1178

merged 1 commit into from
Jul 18, 2022

Conversation

corehalt
Copy link
Contributor

@corehalt corehalt commented Jul 12, 2022

Compliance tests are not in sync with the logs generated by the performance/accuracy runs.
Without these, the tests cannot find the estimated value and it returns zero for both and the tests will silently pass.
Possibly required for r2.1.

@github-actions
Copy link

github-actions bot commented Jul 12, 2022

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

@nvpohanh
Copy link
Contributor

@rnaidu02 @ashwin Could we ask WG to merge this as soon as possible? Otherwise, all SingleStream/MultiStream compliance tests are failing. Thanks

@rnaidu02 rnaidu02 merged commit 67eb6a0 into mlcommons:r2.1 Jul 18, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Jul 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants