Skip to content

Conversation

@rgasper
Copy link
Contributor

@rgasper rgasper commented Oct 10, 2025

📝 Summary

just add inference profiles to existing bedrock models, and add the new ones

fixes #6675

solution simplified after discussion on #6695

Note that I couldn't actually properly test this, because marimo seemingly doesn't currently detect or properly implement AWS CLI SSO authorization when using the pre-configured provider - and I only have AWS SSO credentials at $JOB. it only seems to work if you use a custom model. I'm not sure what I changed in #6695 that affected provider authorization, but that was somehow fixed in that PR.

🔍 Description of Changes

self-evident?

📋 Checklist

  • I have read the contributor guidelines.
  • For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).
  • I have added tests for the changes made.
  • I have run the code and verified that it works as expected.

@vercel
Copy link

vercel bot commented Oct 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
marimo-docs Ready Ready Preview Comment Oct 10, 2025 10:31pm

@mscolnick mscolnick merged commit e07384b into marimo-team:main Oct 10, 2025
21 checks passed
@dmadisetti dmadisetti added the enhancement New feature or request label Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bedrock Models don't work by default

3 participants