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

various updates for CM tools for Windows and better debugging #316

Open
wants to merge 27 commits into
base: main
Choose a base branch
from

Conversation

gfursin
Copy link
Contributor

@gfursin gfursin commented Oct 1, 2024

No description provided.

@gfursin gfursin requested a review from a team as a code owner October 1, 2024 12:36
Copy link

github-actions bot commented Oct 1, 2024

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

@gfursin
Copy link
Contributor Author

gfursin commented Oct 1, 2024

@anandhu-eng - I updated Windows tools with the newest wget.exe as you suggested:

@gfursin
Copy link
Contributor Author

gfursin commented Oct 1, 2024

I turned on test on Windows to make sure that the core CM functionality is not broken.

@arjunsuresh
Copy link
Contributor

@gfursin Windows test won't work with MLPerf as the default workflow is doing llvm installation for loadgen build which works only with user interaction.

gfursin and others added 2 commits October 1, 2024 17:39
* added windows test for image classification
* removed windows test for MLPerf (requires interaction)
@gfursin
Copy link
Contributor Author

gfursin commented Oct 1, 2024

@gfursin Windows test won't work with MLPerf as the default workflow is doing llvm installation for loadgen build which works only with user interaction.

Oh, that's correct! Thanks @arjunsuresh! I removed windows test for MLPerf and added for python image-classification example with ONNX (it is my usual non-interactive test on Windows that touches many CM scripts).

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.

3 participants