Skip to content

Update prompt and unify reasoning budged and enable thinking#386

Open
nv-pranjald wants to merge 4 commits intorelease-v2.5.0from
dev/pranjald/prompt-tuining
Open

Update prompt and unify reasoning budged and enable thinking#386
nv-pranjald wants to merge 4 commits intorelease-v2.5.0from
dev/pranjald/prompt-tuining

Conversation

@nv-pranjald
Copy link
Collaborator

@nv-pranjald nv-pranjald commented Feb 26, 2026

Description

Checklist

  • I am familiar with the Contributing Guidelines.
  • All commits are signed-off (git commit -s) and GPG signed (git commit -S).
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.
  • If adjusting docker-compose.yaml environment variables have you ensured those are mimicked in the Helm values.yaml file.

@nv-pranjald nv-pranjald self-assigned this Feb 26, 2026
@nv-pranjald nv-pranjald changed the title Dev/pranjald/prompt tuining Update prompt and unify reasoning budged and enable thinking Feb 26, 2026
@nv-pranjald nv-pranjald force-pushed the dev/pranjald/prompt-tuining branch from 188d0b3 to dc6a458 Compare February 27, 2026 10:38
# Set to "false" to disable reasoning and get direct answers
ENABLE_NEMOTRON_3_NANO_THINKING: ${ENABLE_NEMOTRON_3_NANO_THINKING:-true}

# Reasoning configuration (supported by Nemotron 3 and other reasoning models)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Need to add a note for this in reasoning doc to point these out.

@shubhadeepd shubhadeepd force-pushed the dev/pranjald/prompt-tuining branch from 699b4dd to 57f3756 Compare February 27, 2026 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants