Skip to content

Short model mapping syntax#1967

Merged
jvstme merged 1 commit intomasterfrom
issue_1590_short_model_mapping
Nov 6, 2024
Merged

Short model mapping syntax#1967
jvstme merged 1 commit intomasterfrom
issue_1590_short_model_mapping

Conversation

@jvstme
Copy link
Copy Markdown
Collaborator

@jvstme jvstme commented Nov 5, 2024

  • Allow to define models in service configurations with a shorter model: <name> syntax.
  • Upgrade docs and examples to the new syntax.
  • Make everything related to format: tgi less visible in the docs, since the default and recommended format is now OpenAI.
  • Show all model mapping fields in the reference, not just those from BaseChatModel.

Closes #1590

- Allow to define models in service configurations
  with a shorter `model: <name>` syntax.
- Upgrade docs and examples to the new syntax.
- Make everything related to `format: tgi` less
  visible in the docs, since the default and
  recommended format is now OpenAI.
- Show all model mapping fields in the reference,
  not just those from `BaseChatModel`
@jvstme jvstme requested a review from un-def November 5, 2024 22:18
@jvstme jvstme merged commit 7f68e94 into master Nov 6, 2024
@jvstme jvstme deleted the issue_1590_short_model_mapping branch November 6, 2024 08:30
superprat pushed a commit to bahaal-tech/dstack that referenced this pull request Dec 20, 2024
- Allow to define models in service configurations
  with a shorter `model: <name>` syntax.
- Upgrade docs and examples to the new syntax.
- Make everything related to `format: tgi` less
  visible in the docs, since the default and
  recommended format is now OpenAI.
- Show all model mapping fields in the reference,
  not just those from `BaseChatModel`.
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.

[UX] Support a shorter syntax for model mapping

2 participants