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

--docker_cm_repo not supporting github links #103

Closed
anandhu-eng opened this issue Jul 9, 2024 · 3 comments
Closed

--docker_cm_repo not supporting github links #103

anandhu-eng opened this issue Jul 9, 2024 · 3 comments

Comments

@anandhu-eng
Copy link
Contributor

To reproduce:

 cm run script --tags=run-mlperf,inference,_find-performance,_full    --model=llama2-70b-99    --implementation=nvidia    --framework=tensorrt    --category=edge    --scenario=Offline    --execution_mode=test    --device=cuda     --docker --quiet    --test_query_count=50 --nvidia_llama2_dataset_file_path=/mnt/llama2/inference/inference/open_orca/open_orca_gpt4_tokenized_llama.sampled_24576.pkl --tp_size=1 --docker_cm_repo=https://github.com/anandhu-eng/cm4mlops.git

Error message:

 > [10/12] RUN cm pull repo https://github.com/anandhu-eng/cm4mlops.git:
1.141 Cloning into 'mlcommons@https://github.com/anandhu-eng/cm4mlops.git'...
1.566 remote: Not Found
1.566 fatal: repository 'https://github.com/mlcommons/https://github.com/anandhu-eng/cm4mlops.git/' not found
1.572
1.572 CM error: repository was not cloned!
1.572 Warning: CM index is used for the first time. CM will reindex all artifacts now - it may take some time ...
1.572 =======================================================
1.572 Alias:    mlcommons@https://github.com/anandhu-eng/cm4mlops.git
1.572 URL:      https://github.com/mlcommons/https://github.com/anandhu-eng/cm4mlops.git
1.572
1.572 Local path: /home/cmuser/CM/repos/mlcommons@https://github.com/anandhu-eng/cm4mlops.git
1.572
1.572 git clone https://github.com/mlcommons/https://github.com/anandhu-eng/cm4mlops.git mlcommons@https://github.com/anandhu-eng/cm4mlops.git
1.572
------
mlperf-inference:mlpinf-v4.0-cuda12.2-cudnn8.9-x86_64-ubuntu20.04-public.Dockerfile:39
--------------------
  37 |
  38 |     # Download CM repo for scripts
  39 | >>> RUN cm pull repo https://github.com/anandhu-eng/cm4mlops.git
  40 |
  41 |     # Install all system dependencies
--------------------
ERROR: failed to solve: process "/bin/bash -c cm pull repo https://github.com/anandhu-eng/cm4mlops.git" did not complete successfully: exit code: 1

CM error: Portable CM script failed (name = build-docker-image, return code = 256)
@arjunsuresh arjunsuresh changed the title --docker_cm_repo no supporting github links --docker_cm_repo not supporting github links Jul 10, 2024
arjunsuresh added a commit that referenced this issue Sep 5, 2024
added sample demo file for cm-sut-info + file rename for consistancy
anandhu-eng added a commit to anandhu-eng/cm4mlops that referenced this issue Sep 30, 2024
@gfursin
Copy link
Contributor

gfursin commented Sep 30, 2024

Hi @anandhu-eng . Was it fixed? I will need this functionality and may try to check and fix it...

@anandhu-eng
Copy link
Contributor Author

Hi @gfursin , sorry I missed the comment. It was fixed in this PR. Please do check if there is any problem. I'm keeping this issue open till then.

@gfursin
Copy link
Contributor

gfursin commented Oct 3, 2024

Cool! No worries @anandhu-eng and thanks! I actually thought that it was working ;) . I close the issue.

@gfursin gfursin closed this as completed Oct 3, 2024
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

No branches or pull requests

2 participants