-
Notifications
You must be signed in to change notification settings - Fork 396
Rename aiq namespace to nat
#618
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
Rename aiq namespace to nat
#618
Conversation
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Update mocks in tests Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
…avid-aiq-prefix-rename Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
…avid-rename-rename-rename Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
…porter.py Co-authored-by: Will Killian <2007799+willkill07@users.noreply.github.com> Signed-off-by: David Gardner <96306125+dagardner-nv@users.noreply.github.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
…AIQtoolkit into david-rename-rename-rename Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
…avid-rename-rename-rename Signed-off-by: David Gardner <dagardner@nvidia.com>
Co-authored-by: Will Killian <2007799+willkill07@users.noreply.github.com> Signed-off-by: David Gardner <96306125+dagardner-nv@users.noreply.github.com>
…yst_exporter.py Co-authored-by: Will Killian <2007799+willkill07@users.noreply.github.com> Signed-off-by: David Gardner <96306125+dagardner-nv@users.noreply.github.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
…AIQtoolkit into david-rename-rename-rename Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
The pull request renames the Python API aiq namespace to nat and the aiq CLI tool to nat across the entire codebase while maintaining compatibility aliases. This includes updating imports, package names, entry points, and file paths throughout the project.
- Rename Python API namespace from
aiqtonat - Rename CLI tool from
aiqtonat - Update all imports, package names, and example project structures
Reviewed Changes
Copilot reviewed 200 out of 954 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| nat.code-workspace | Updates VS Code project configuration to disable Python language server |
| packages/nvidia_nat_agno/ | Renames Agno integration package imports and entry points |
| examples/ | Comprehensive update of all example packages to use nat namespace |
| docs/source/ | Updates documentation references from aiq to nat |
Signed-off-by: David Gardner <dagardner@nvidia.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: David Gardner <96306125+dagardner-nv@users.noreply.github.com>
… the pypi package name Signed-off-by: David Gardner <dagardner@nvidia.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor feedback
Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Co-authored-by: Will Killian <2007799+willkill07@users.noreply.github.com> Signed-off-by: David Gardner <96306125+dagardner-nv@users.noreply.github.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
Co-authored-by: Will Killian <2007799+willkill07@users.noreply.github.com> Signed-off-by: David Gardner <96306125+dagardner-nv@users.noreply.github.com>
Co-authored-by: Will Killian <2007799+willkill07@users.noreply.github.com> Signed-off-by: David Gardner <96306125+dagardner-nv@users.noreply.github.com>
* Rename the Python API `aiq` namespace to `nat` * Rename the `aiq` CLI tool to `nat` * Leaving a compatibility alias in place for both. * Rename the VSCode project name * Rename the `aiq_` prefix in examples code paths * Pylint is currently failing due to this PR touching all Python files, other checks (isort, flake8, yapf, uv-lock and link check) are passing, as are the unittests. * Rename submodule `external/aiqtoolkit-opensource-ui` -> `external/nat-ui` * Rename default `output_dir` in example YAMLs from `.tmp/aiq/...` to `.tmp/nat/...` Things not in this PR to be handled in another PR: * Documentation updates * Docstring updates - I am familiar with the [Contributing Guidelines](https://github.com/NVIDIA/NeMo-Agent-Toolkit/blob/develop/docs/source/resources/contributing.md). - We require that all contributors "sign-off" on their commits. This certifies that the contribution is your original work, or you have rights to submit it under the same license, or a compatible license. - Any contribution which contains commits that are not Signed-Off will not be accepted. - When the PR is ready for review, new or existing tests cover these changes. - When the PR is ready for review, the documentation is up to date with these changes. --------- Signed-off-by: David Gardner <dagardner@nvidia.com> Signed-off-by: David Gardner <96306125+dagardner-nv@users.noreply.github.com> Co-authored-by: Will Killian <2007799+willkill07@users.noreply.github.com> Co-authored-by: Yuchen Zhang <134643420+yczhang-nv@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Description
aiqnamespace tonataiqCLI tool tonataiq_prefix in examples code pathsexternal/aiqtoolkit-opensource-ui->external/nat-uioutput_dirin example YAMLs from.tmp/aiq/...to.tmp/nat/...Things not in this PR to be handled in another PR:
By Submitting this PR I confirm: