Skip to content

Conversation

@Chrisischris
Copy link

Implements the extra_args feature for /models/load that is already documented in the server README but was not yet implemented.

From the docs:

extra_args: (optional) an array of additional arguments to be passed to the model instance. Note: you must start the server with --models-allow-extra-args to enable this feature.

Changes

  • common/common.h: Add models_allow_extra_args param
  • common/arg.cpp: Add --models-allow-extra-args / --no-models-allow-extra-args flag
  • tools/server/server-models.h: Update load() signature to accept extra_args
  • tools/server/server-models.cpp: Parse extra_args from JSON body and append to child process args

See server README for usage documentation.

@github-actions github-actions bot added the devops improvements to build systems and github actions label Dec 21, 2025
@Chrisischris Chrisischris marked this pull request as ready for review December 21, 2025 17:48
@Chrisischris Chrisischris marked this pull request as draft December 21, 2025 17:49
Comment on lines -184 to +188
- build: 's390x'
os: ubuntu-24.04-s390x
- build: 'ppc64le'
os: ubuntu-24.04-ppc64le
# Disabled - requires special GitHub runners not available on forks
# - build: 's390x'
# os: ubuntu-24.04-s390x
# - build: 'ppc64le'
# os: ubuntu-24.04-ppc64le
Copy link
Collaborator

Choose a reason for hiding this comment

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

Restore please.

Comment on lines -137 to +139
- build: 's390x'
os: ubuntu-24.04-s390x
# Disabled - requires special GitHub runners not available on forks
# - build: 's390x'
# os: ubuntu-24.04-s390x
Copy link
Collaborator

Choose a reason for hiding this comment

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

Restore please.

Copy link
Author

Choose a reason for hiding this comment

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

here created a new PR #18261

Copy link
Collaborator

Choose a reason for hiding this comment

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

Not necessary, but ok...

@Chrisischris
Copy link
Author

Closing to recreate from a clean branch without unrelated CI changes.

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

Labels

devops improvements to build systems and github actions examples server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants