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: Add Google Gemini provider #79

Merged
merged 1 commit into from
Feb 9, 2025
Merged

feat: Add Google Gemini provider #79

merged 1 commit into from
Feb 9, 2025

Conversation

lpgn
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.

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
Copy link
Owner

djcopley commented Feb 8, 2025

closes #78

@djcopley
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
@suizideFloat
Copy link

works 🎉

@lpgn
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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants