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

Closed
wants to merge 0 commits into from

Conversation

twilwa
Copy link
Collaborator

@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
Collaborator 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 twilwa force-pushed the minor-linux-fixes branch from 58aa5d2 to 1c9d277 Compare November 4, 2024 21:08
@twilwa
Copy link
Collaborator 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

@twilwa twilwa closed this Nov 6, 2024
@twilwa twilwa force-pushed the minor-linux-fixes branch from 1c9d277 to 39ce28e Compare November 6, 2024 22:20
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