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

Add an important note to master #292

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bitfort
Copy link
Contributor

@bitfort bitfort commented Mar 3, 2023

Most MLPerf repositories use master as their source of truth. We should either migrate to that or clearly indicate not to use this branch.

Most MLPerf repositories use master as their source of truth. We should either migrate to that or clearly indicate not to use this branch.
@github-actions
Copy link

github-actions bot commented Mar 3, 2023

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

@arjunsuresh
Copy link
Contributor

Thank you @bitfort for this suggestion. Actually our plan was indeed to use the master branch for submission until this issue came and the related emergency meeting which necessitated a move to r3.0 branch for power. This was announced in the meetings (both power and inference) and also the power-dev git submodule of the inference repository was updated to r3.0 branch. But I understand that such announcements may not be reaching each and every developer working for different submitters and we have seen this happen previously too. In fact this is precisely why inference WG moved to using the master branch for submission and power-dev can also do the same from next rounds. But even then any submitter must be using a valid commit hash (or later) and I guess this check we can add as soon as we have that valid commit hash for 3.1 and so on.

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