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

Adds limited Gemma support, disables secondhand knowledge again #51

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zmerlynn
Copy link
Collaborator

  • Adds a Gemma deployment, but it needs an API key. I would like to add a secret, probably in common, that has to be edited before deployment, so I'm not advertising it in the README yet.

  • After Optimize Spanner queries #46, secondhand knowledge was reenabled. Disable for now - this needs more prompt work. I tried a couple of things to improve it that are included in this PR but ultimately disabled it again.

  • Update TGI to 1.4.4

* Adds a Gemma deployment, but it needs an API key. I would like to
add a secret, probably in common, that has to be edited before
deployment, so I'm not advertising it in the README yet.

* After googleforgames#46, secondhand knowledge was reenabled. Disable for now -
this needs more prompt work. I tried a couple of things to improve it
that are included in this PR but ultimately disabled it again.

* Update TGI to 1.4.4
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