Skip to content

feat(bedrock): Record inferenceConfig parameters - #8232

Draft
alexander-alderman-webb wants to merge 1 commit into
masterfrom
webb/bedrock/request-parameters
Draft

alexander-alderman-webb wants to merge 1 commit into
masterfrom
webb/bedrock/request-parameters

Conversation

@alexander-alderman-webb

Copy link
Copy Markdown
Collaborator

Set the maximum output tokens, temperature and top p request parameters from the Bedrock API.

@alexander-alderman-webb
alexander-alderman-webb force-pushed the webb/bedrock/request-parameters branch from e18dff0 to 9c7ab68 Compare November 9, 2026 14:25
@alexander-alderman-webb alexander-alderman-webb changed the title feat(bedrock): Record request parameters feat(bedrock): Record inferenceConfig parameters Nov 9, 2026
@github-actions

github-actions Bot commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

Coverage Results 📊

✅ 137283 passed | ⏭️ 7575 skipped | Total: 144858 | Pass Rate: 94.77% | Execution Time: 446m 9s

📊 Comparison with Base Branch

Metric Change
Total Tests —
Passed Tests —
Failed Tests —
Skipped Tests —

✨ Test counts unchanged from base.

All tests are passing successfully.

✅ Patch coverage is 92.86% (13 of 14 changed executable lines covered; target 80%).
Project statement coverage is 90.29% (down 0.01 percentage points from base (da37f0a) to head (9c7ab68)).

Changed files with executable lines (1)
File Patch coverage Changed executable lines
sentry_sdk/integrations/boto3/_services/bedrock.py 92.86% 13/14 covered; missed: 40; partial branches: 39, 43, 47, 51
Coverage diff
@@            Coverage Diff             @@
##        master     #8232       +/-##
==========================================
- Coverage    90.30%    90.29%    -0.01%
==========================================
  Files          203       203         —
  Tracked lines     26953     26966       +13
  Branches     10066     10074        +8
==========================================
+ Hits         24338     24348       +10
- Misses        2615      2618        +3
- Partials      1520      1526        +6

Generated by Coverage Action

This branch has not been deployed

No deployments
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.

1 participant