Skip to content

Use opus 4.7 for reviews#6598

Merged
petebacondarwin merged 2 commits into
mainfrom
dlapid/opus47
Apr 17, 2026
Merged

Use opus 4.7 for reviews#6598
petebacondarwin merged 2 commits into
mainfrom
dlapid/opus47

Conversation

@danlapid

Copy link
Copy Markdown
Collaborator

No description provided.

@danlapid danlapid requested review from a team as code owners April 16, 2026 22:48
@danlapid danlapid requested a review from ascorbic April 16, 2026 22:48
@ask-bonk

ask-bonk Bot commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

ProviderModelNotFoundError

github run

@ask-bonk

ask-bonk Bot commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

@danlapid Bonk workflow failed. Check the logs for details.

View workflow run · To retry, trigger Bonk again.

@ask-bonk

ask-bonk Bot commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

ProviderModelNotFoundError

github run

@danlapid

Copy link
Copy Markdown
Collaborator Author

Depends on anomalyco/models.dev#1472

@codspeed-hq

codspeed-hq Bot commented Apr 16, 2026

Copy link
Copy Markdown

Merging this PR will degrade performance by 12.7%

❌ 1 regressed benchmark
✅ 71 untouched benchmarks
⏩ 129 skipped benchmarks1

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
Encode_ASCII_32[TextEncoder][0/0/32] 2.7 ms 3.1 ms -12.7%

Comparing dlapid/opus47 (a748981) with main (d1e13ad)

Open in CodSpeed

Footnotes

  1. 129 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@ask-bonk

ask-bonk Bot commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

ProviderModelNotFoundError

github run

@petebacondarwin petebacondarwin merged commit 251a80b into main Apr 17, 2026
28 of 31 checks passed
@petebacondarwin petebacondarwin deleted the dlapid/opus47 branch April 17, 2026 22:26
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.

4 participants