Skip to content

feat(nova-evaluation): Add checkpoint_is_instruct_tuned to InferenceConfig#374

Open
amaz-jmoul wants to merge 1 commit intoaws:release_v2from
amaz-jmoul:add-checkpoint-instruct-tuned-param
Open

feat(nova-evaluation): Add checkpoint_is_instruct_tuned to InferenceConfig#374
amaz-jmoul wants to merge 1 commit intoaws:release_v2from
amaz-jmoul:add-checkpoint-instruct-tuned-param

Conversation

@amaz-jmoul
Copy link

What's changing and why?

This parameter is used during evaluation runs on CPT models. We need to add it here so validations pass when using SMHP.
A customer tried to use this parameter which is supported by the evaluation container but got a validation error because it is not present here.

Before/After UX

Before:
Customer tries to use the new checkpoint_is_instruct_tuned = "false" parameter, but failed due to pydantic validation error since it's not here.

After:
Customer is able to set checkpoint_is_instruct_tuned = "false" and run CPT evaluation.

How was this change tested?

We have verified that the checkpoint_is_instruct_tuned parameter works with the evaluation container internally.

Are unit tests added?

No

Are integration tests added?

No

Reviewer Guidelines

‼️ Merge Requirements: PRs with failing integration tests cannot be merged without justification.

One of the following must be true:

  • All automated PR checks pass
  • Failed tests include local run results/screenshots proving they work
  • [] Changes are documentation-only

@amaz-jmoul amaz-jmoul requested a review from a team as a code owner February 18, 2026 22:13
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.

2 participants

Comments