Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Integrate OpenRouter as model provider #193

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

twilwa
Copy link
Contributor

@twilwa twilwa commented Nov 4, 2024

Don't think there's an issue open for openrouter, but this technically works. I'd like to clean it up a bit as I'm not certain it's priortizing correctly (character.json, .env, fallback to oai) at all steps and I haven't tested trying to change the small/med/large model at those steps, but thought I'd toss it up in case we wanted a 'fuck it, shove it in anyway' type beat lol

Relates to:

Risks

Background

What does this PR do?

What kind of change is this?

Documentation changes needed?

Testing

Where should a reviewer start?

Detailed testing steps

@lalalune
Copy link
Member

lalalune commented Nov 4, 2024

LGTM just --

  1. rebase on new, run the prettier
  2. delete the unnecessary files
Screenshot 2024-11-03 at 9 55 08 PM

@twilwa
Copy link
Contributor Author

twilwa commented Nov 4, 2024

LGTM just --

  1. rebase on new, run the prettier
  2. delete the unnecessary files
Screenshot 2024-11-03 at 9 55 08 PM

actually havent done a rebase berfore -- seems like it's walking through every commit on main since i opened the branch to check for conflicts, is there a way to mass-accept incoming rebase changes? otherwise 400+ 'approve me' steps

think i fixed git so its not adding my weird extra files tho

@twilwa
Copy link
Contributor Author

twilwa commented Nov 5, 2024

mk, not exaaactly sure what i did but i think that broke everything lol. thankfully still have the cursor convo, so will try to fix and then maybe try and pair with someone to finish

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