Skip to content

Conversation

@whoisj
Copy link
Contributor

@whoisj whoisj commented Dec 8, 2025

This change improves the model management documentation.

Explicitly explains that models should be unloaded before being reloaded when using EXPLICIT mode.

TRI-249

This change improves the model management documentation.

Explicitily explains that models should be unloaded before being reloaded when using EXPLICIT mode.
@whoisj whoisj requested a review from mattwittwer December 9, 2025 00:39
- Changes to the model repository while the server is running will be ignored.
Model load and unload requests using the [model control protocol](../protocol/extension_model_repository.md) will have no affect and will return an error response.

- This model control mode is selected by specifying `--model-control-mode=none` when starting Triton.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- This model control mode is selected by specifying `--model-control-mode=none` when starting Triton.
- This model control mode is selected by specifying `--model-control-mode` when starting Triton.

This model control mode is selected by specifying `--model-control-mode=none` when starting Triton.
This is the default model control mode.
Changing the model repository while Triton is running must be done carefully, as explained in [Modifying the Model Repository](#modifying-the-model-repository).
- This is the default model control mode.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- This is the default model control mode.
- The default model control mode is `--model-control-mode=none`.

- This is the default model control mode.

> [!IMPORTANT]
> Changing the model repository while Triton is running must be done carefully, as explained in [Modifying the Model Repository](#modifying-the-model-repository).
Copy link
Contributor

Choose a reason for hiding this comment

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

Should remove duplicated warnings in the following sections.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants