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

Should summary.xlsx be included in this repo or generated dynamically? #30

Closed
gfursin opened this issue Oct 8, 2024 · 1 comment
Closed

Should summary.xlsx be included in this repo or generated dynamically? #30

gfursin opened this issue Oct 8, 2024 · 1 comment
Assignees
@arjunsuresh @gfursin @anandhu-eng
Labels
enhancement New feature or request

Comments

@gfursin
Copy link

gfursin commented Oct 8, 2024

I think we may have merge issues if summary.xlsx is updated in different repositories and then pushed here ... Maybe the best solution is to remove it from here and generate dynamically at MLCommons website? Any thoughts?
@gfursin gfursin added the enhancement New feature or request label Oct 8, 2024
@gfursin gfursin mentioned this issue Oct 8, 2024
Merged
@arjunsuresh
Copy link
Contributor

yes @gfursin It is currently working like that. Being a binary file it was creating git conflicts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arjunsuresh arjunsuresh

@gfursin gfursin

@anandhu-eng anandhu-eng

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@arjunsuresh @gfursin @anandhu-eng