Skip to content

Conversation

nullfunc
Copy link
Contributor

@nullfunc nullfunc commented May 20, 2025

Update to remove some setting from compose. Use mode Id from Amazon for meaningful model name.

Samples Checklist

✅ All good!

@nullfunc nullfunc had a problem deploying to deploy-changed-samples May 20, 2025 22:57 — with GitHub Actions Failure
environment:
- ENDPOINT_URL=http://llm/api/v1/chat/completions # endpoint to the gateway service
- MODEL=anthropic.claude-3-haiku-20240307-v1:0 # LLM model ID used for the gateway
- MODEL=us.amazon.nova-micro-v1:0 # LLM model ID used for the gateway
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't we use a cloud-agnostic model?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not for this. This is a AWS playground only sample so it makes more sense to be clear that the model is AWS only. We will abstract this at the next step where we have a model on all platforms (likely llama or mistral).

Copy link
Member

Choose a reason for hiding this comment

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

ai/llama3.2 is available on all clouds right? Why not use that?

Copy link
Member

Choose a reason for hiding this comment

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

I am skeptical that an "AWS playground only sample" is reasonable

@nullfunc
Copy link
Contributor Author

closing in favour of #378

@nullfunc nullfunc closed this May 20, 2025
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