Skip to content

Removing pip install instruction for NeMo voice agent environment setting#15101

Merged
tango4j merged 3 commits intoNVIDIA-NeMo:mainfrom
tango4j:va_env_setting_rev
Dec 15, 2025
Merged

Removing pip install instruction for NeMo voice agent environment setting#15101
tango4j merged 3 commits intoNVIDIA-NeMo:mainfrom
tango4j:va_env_setting_rev

Conversation

@tango4j
Copy link
Copy Markdown
Collaborator

@tango4j tango4j commented Nov 20, 2025

What does this PR do ?

Since NeMo voice agent is not pip install friendly, we are removing the pip-install instructions.

Collection: [Note which collection this PR will affect]

NeMo voice agent

Changelog

Remove the few lines in examples/voice_agent/README.md

Usage

See https://github.com/NVIDIA-NeMo/NeMo/tree/main/examples/voice_agent

Before your PR is "Ready for review"

Pre checks:

  • Make sure you read and followed Contributor guidelines
  • Did you write any new necessary tests?
  • Did you add or update any necessary documentation?
  • Does the PR affect components that are optional to install? (Ex: Numba, Pynini, Apex etc)
    • Reviewer: Does the PR have correct import guards for all optional libraries?

PR Type:

  • New Feature
  • Bugfix
  • Documentation

If you haven't finished some of the above items you can still open "Draft" PR.

Who can review?

Anyone in the community is free to review the PR once the checks have passed.
Contributor guidelines contains specific people who can review PRs to various areas.

Additional Information

  • Related to # (issue)

Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: taejinp <tango4j@gmail.com>
@tango4j tango4j requested a review from stevehuang52 November 20, 2025 19:46
@tango4j
Copy link
Copy Markdown
Collaborator Author

tango4j commented Nov 20, 2025

@chtruong814
This is document change PR so it can skip tests.

@tango4j tango4j marked this pull request as ready for review December 5, 2025 18:54
Copy link
Copy Markdown
Collaborator

@KunalDhawan KunalDhawan left a comment

Choose a reason for hiding this comment

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

LGTM, conditionally approved. This is required to ensure that 'nemo-toolkit[all]' installation doesn't conflict with the pinned library version in environment.yaml. However, we need to add nemo-toolkit support later to ensure that all nemotron models are supported.

@tango4j tango4j added the r2.6.0 Cherry-pick label for the r2.6.0 release branch label Dec 6, 2025
@tango4j tango4j enabled auto-merge (squash) December 6, 2025 01:54
@tango4j tango4j added Run CICD and removed Run CICD labels Dec 15, 2025
@tango4j tango4j merged commit 88892b5 into NVIDIA-NeMo:main Dec 15, 2025
64 of 70 checks passed
AkCodes23 pushed a commit to AkCodes23/NeMo that referenced this pull request Jan 28, 2026
…ting (NVIDIA-NeMo#15101)

* Removing pip install for Nemotron inclusion

Signed-off-by: taejinp <tango4j@gmail.com>

* Removing requirements file

Signed-off-by: taejinp <tango4j@gmail.com>

---------

Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: Akhil Varanasi <akhilvaranasi23@gmail.com>
nune-tadevosyan pushed a commit to nune-tadevosyan/NeMo that referenced this pull request Mar 13, 2026
…ting (NVIDIA-NeMo#15101)

* Removing pip install for Nemotron inclusion

Signed-off-by: taejinp <tango4j@gmail.com>

* Removing requirements file

Signed-off-by: taejinp <tango4j@gmail.com>

---------

Signed-off-by: taejinp <tango4j@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

r2.6.0 Cherry-pick label for the r2.6.0 release branch Run CICD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants