Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat: Add Google Gemini provider #79

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

Merged
djcopley merged 1 commit into djcopley:main from lpgn:main
Feb 9, 2025
Merged

feat: Add Google Gemini provider #79

djcopley merged 1 commit into djcopley:main from lpgn:main
Feb 9, 2025

Conversation

Copy link
Contributor

@lpgn lpgn commented Feb 7, 2025

This commit adds a new provider for Google Gemini, allowing users to select Gemini as their LLM provider in ShellOracle.

The following changes were made:

  • Created src/shelloracle/providers/google.py with the Google provider class.
  • Added google-generativeai as a dependency in pyproject.toml.
  • Modified src/shelloracle/providers/__init__.py to include the new provider.
  • Updated ~/.shelloracle/config.toml to include a configuration section for the Google provider.

djcopley reacted with hooray emoji
This commit adds a new provider for Google Gemini, allowing users to select Gemini as their LLM provider in ShellOracle.
The following changes were made:
- Created `src/shelloracle/providers/google.py` with the `Google` provider class.
- Added `google-generativeai` as a dependency in `pyproject.toml`.
- Modified `src/shelloracle/providers/__init__.py` to include the new provider.
- Updated `~/.shelloracle/config.toml` to include a configuration section for the Google provider.
Copy link
Owner

djcopley commented Feb 8, 2025

closes #78

Copy link
Owner

djcopley commented Feb 8, 2025

Thanks for your PR! Interesting the gemini requires a model response before user input. Any idea why that is?

@djcopley djcopley merged commit 8efabfe into djcopley:main Feb 9, 2025
5 of 6 checks passed
Copy link

works 🎉

Copy link
Contributor Author

lpgn commented Feb 10, 2025

Thanks for your PR! Interesting the gemini requires a model response before user input. Any idea why that is?

it doesn ́t I was probably hallucinating :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /