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

code clean - qaic software kit gh action + resnet50 gh action update #146

Merged
merged 9 commits into from
Oct 6, 2024

Conversation

anandhu-eng
Copy link

No description provided.

@anandhu-eng anandhu-eng changed the title code clean code clean - qaic software kit gh action Oct 5, 2024
@anandhu-eng anandhu-eng changed the title code clean - qaic software kit gh action code clean - qaic software kit gh action + resnet50 gh action update Oct 5, 2024
- name: Test MLPerf Inference MLCommons C++ ResNet50 on ${{ matrix.os }}
if: matrix.os == 'windows-latest'
run: |
cmr "app mlperf inference mlcommons cpp" --submitter="MLCommons" --hw_name=gh_${{ matrix.os }} --adr.loadgen.tags=_from-pip --pip_loadgen=yes -v --quiet

Choose a reason for hiding this comment

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

Is pip install of loadgen enough to use loadgen in C++? Also, we don't have an automatic way to install C++ compiler on windows right?

Choose a reason for hiding this comment

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

I think we can install llvm on Linux using
choco install llvm

Copy link
Author

Choose a reason for hiding this comment

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

I think we can install llvm on Linux using choco install llvm

Hi @arjunsuresh , did you mean on windows? Also, when using Chocolatey on Windows, do users need to run the command in PowerShell or Command Prompt opened with administrative privileges?

Choose a reason for hiding this comment

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

@anandhu-eng I'm not sure. But no need to worry about it now - we don't need to support windows.

@anandhu-eng anandhu-eng marked this pull request as draft October 5, 2024 17:54
@arjunsuresh arjunsuresh marked this pull request as ready for review October 6, 2024 17:52
@arjunsuresh arjunsuresh merged commit d8921ed into GATEOverflow:mlperf-inference Oct 6, 2024
34 checks passed
@anandhu-eng anandhu-eng deleted the rnnthaction branch October 6, 2024 18:00
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