Skip to content

Conversation

visheshtanksale
Copy link
Collaborator

No description provided.

Copy link

copy-pr-bot bot commented Sep 4, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

env:
- name: NIM_MODEL_NAME
value: nvidia/nemo/llama-3_2-1b-instruct
value: 'ngc://nvidian/nim-llm-dev/meta-llama3-8b-instruct:hf'
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is internal NGC repository. Let's remove it for now? thanks

env:
- name: NIM_MODEL_NAME
value: hf://meta-llama/Llama-3.2-1B-Instruct
- name: HF_TOKEN
Copy link
Collaborator

@shivamerla shivamerla Sep 16, 2025

Choose a reason for hiding this comment

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

This will need to be part of secret as well, in this case we should not make NGC_API_KEY checks in the code mandatory and export all env from the secret instead.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes thats the fix. But this PR is just to fix the sample for v3.0.0 release.

@shivamerla shivamerla marked this pull request as draft October 10, 2025 17:15
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