Skip to content

feat: Added Liner service - #249

Open
FieldMarshallObvious wants to merge 4 commits into
mainfrom
feat/liner-quick-answers-budgeting-AIPLAT-1275
Open

feat: Added Liner service #249
FieldMarshallObvious wants to merge 4 commits into
mainfrom
feat/liner-quick-answers-budgeting-AIPLAT-1275

Conversation

@FieldMarshallObvious

@FieldMarshallObvious FieldMarshallObvious commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

What this PR does

Adds liner-answer service type, and restricts liner-answers model to only be allowed on that service type. New configurations where also added to support liner budgeting defaults, and liner-api-keys. Unit tests where added to cover the new set of cases for exa.

QA Path

Verified on local sending requests with liner-answers model on liner-answer` service type and alternative service types.

Valid Service Type:
image

Invalid Service Type:
image

Comment thread src/tests/unit/test_config.py Outdated
Comment thread src/mlpa/core/config.py Outdated
Comment thread litellm_config.yaml Outdated
# Note this model requires a custom provider map to handle liner requests
#- model_name: liner-answers
# litellm_params:
# model: liner-answers/liner-quick-answer

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

the indents on these lines when uncommented are 1 space short

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Also maybe add instructions (or commented out line) that to add the volume in litellm_docker_compose.yaml too

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Maybe this is my vim mode deceiving me but they seem to be correct?
image

@noahpodgurski noahpodgurski left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Super awesome that we have this working through LiteLLM now! Just a few comments 🙌

@FieldMarshallObvious
FieldMarshallObvious force-pushed the feat/liner-quick-answers-budgeting-AIPLAT-1275 branch from 4cf3a3c to ee7df47 Compare September 1, 2026 18:43
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