KEMBAR78
Use consistent casing for ReAct agent by dagardner-nv · Pull Request #293 · NVIDIA/NeMo-Agent-Toolkit · GitHub
Skip to content

Conversation

@dagardner-nv
Copy link
Contributor

Description

  • "agent" is lower-case outside of headers
  • Update the default value for description to match that used in src/aiq/agent/react_agent/register.py
  • Fix link to reasoning agent documentation

Closes #261

By Submitting this PR I confirm:

  • I am familiar with the Contributing Guidelines.
  • 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.

… headers. Update the default value to match that used in src/aiq/agent/react_agent/register.py

Signed-off-by: David Gardner <dagardner@nvidia.com>
Signed-off-by: David Gardner <dagardner@nvidia.com>
@dagardner-nv dagardner-nv added doc Improvements or additions to documentation non-breaking Non-breaking change labels May 16, 2025
@dagardner-nv dagardner-nv self-assigned this May 16, 2025
@dagardner-nv
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 6107c27 into NVIDIA:develop May 16, 2025
12 checks passed
@dagardner-nv dagardner-nv deleted the david-react-agent-doc-261 branch May 16, 2025 21:31
ericevans-nv pushed a commit to ericevans-nv/agent-iq that referenced this pull request Jun 3, 2025
* "agent" is lower-case outside of headers
* Update the default value for `description` to match that used in `src/aiq/agent/react_agent/register.py`
* Fix link to reasoning agent documentation

Closes NVIDIA#261

## By Submitting this PR I confirm:
- I am familiar with the [Contributing Guidelines](https://github.com/NVIDIA/AIQToolkit/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.

Authors:
  - David Gardner (https://github.com/dagardner-nv)

Approvers:
  - Matthew Penn (https://github.com/mpenn)
  - https://github.com/lvojtku

URL: NVIDIA#293
Signed-off-by: Eric Evans <194135482+ericevans-nv@users.noreply.github.com>
ericevans-nv pushed a commit to ericevans-nv/agent-iq that referenced this pull request Jun 3, 2025
* "agent" is lower-case outside of headers
* Update the default value for `description` to match that used in `src/aiq/agent/react_agent/register.py`
* Fix link to reasoning agent documentation

Closes NVIDIA#261

## By Submitting this PR I confirm:
- I am familiar with the [Contributing Guidelines](https://github.com/NVIDIA/AIQToolkit/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.

Authors:
  - David Gardner (https://github.com/dagardner-nv)

Approvers:
  - Matthew Penn (https://github.com/mpenn)
  - https://github.com/lvojtku

URL: NVIDIA#293
Signed-off-by: Eric Evans <194135482+ericevans-nv@users.noreply.github.com>
AnuradhaKaruppiah pushed a commit to AnuradhaKaruppiah/oss-agentiq that referenced this pull request Aug 4, 2025
* "agent" is lower-case outside of headers 
* Update the default value for `description` to match that used in `src/aiq/agent/react_agent/register.py`
* Fix link to reasoning agent documentation

Closes NVIDIA#261

## By Submitting this PR I confirm:
- I am familiar with the [Contributing Guidelines](https://github.com/NVIDIA/AIQToolkit/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.

Authors:
  - David Gardner (https://github.com/dagardner-nv)

Approvers:
  - Matthew Penn (https://github.com/mpenn)
  - https://github.com/lvojtku

URL: NVIDIA#293
scheckerNV pushed a commit to scheckerNV/aiq-factory-reset that referenced this pull request Aug 22, 2025
* "agent" is lower-case outside of headers 
* Update the default value for `description` to match that used in `src/aiq/agent/react_agent/register.py`
* Fix link to reasoning agent documentation

Closes NVIDIA#261

## By Submitting this PR I confirm:
- I am familiar with the [Contributing Guidelines](https://github.com/NVIDIA/AIQToolkit/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.

Authors:
  - David Gardner (https://github.com/dagardner-nv)

Approvers:
  - Matthew Penn (https://github.com/mpenn)
  - https://github.com/lvojtku

URL: NVIDIA#293
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Improvements or additions to documentation non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DOC]: Inconsistent casing of ReAct Agent in docs/source/workflows/about/react-agent.md

3 participants