Skip to content

Add Support for Openrouter as API router & Anthropic Claude Sonnet#3

Closed
0xrsydn wants to merge 2 commits intodzhng:mainfrom
0xrsydn:provider
Closed

Add Support for Openrouter as API router & Anthropic Claude Sonnet#3
0xrsydn wants to merge 2 commits intodzhng:mainfrom
0xrsydn:provider

Conversation

@0xrsydn
Copy link
Copy Markdown

@0xrsydn 0xrsydn commented Feb 4, 2025

I tried using claude sonnet via openrouter api and it works great also the report results was quite decent.

Other models on openrouter, I havent try it but the openai one need tier 3 one also for deepseek model integration one especially R1 might need some adjustment

@freezscholte
Copy link
Copy Markdown

what also would be cool to capture the R1 tough part and let another agent review this? just a brain dump was playing around with this idea that it sort of automatically self improves..

@0xrsydn
Copy link
Copy Markdown
Author

0xrsydn commented Feb 4, 2025

what also would be cool to capture the R1 tough part and let another agent review this? just a brain dump was playing around with this idea that it sort of automatically self improves..

yeah i have tried using r1 one, apparently i have to change how the code works too as i have to parse process of r1 also how r1 handle the code (i tried it with openrouter api, think doesnt work on this code especially generateObject one)

@bhupesh-sf
Copy link
Copy Markdown

allow configurable openai base url

@dzhng
Copy link
Copy Markdown
Owner

dzhng commented Feb 5, 2025

I really want to keep this repo simple and <500 LoC so it's easy for people to come in and understand. mind that I don't merge this and just keep this PR open so anyone who wants to do this can get a good reference implementation

pal added a commit to pal/deep-research that referenced this pull request Feb 6, 2025
@dzhng dzhng mentioned this pull request Feb 7, 2025
@dzhng
Copy link
Copy Markdown
Owner

dzhng commented Feb 7, 2025

FYI latest version supports custom endpoints & models via env vars, which also let's you support openrouter (since they're openai api compatible). I think that's cleaner so going to close this PR

@dzhng dzhng closed this Feb 7, 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.

4 participants