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 Chat Message History and Connects to the NPC Chat API #3679

Merged
merged 6 commits into from
Mar 8, 2024

Conversation

igooch
Copy link
Collaborator

@igooch igooch commented Mar 4, 2024

What type of PR is this?
/kind documentation

What this PR does / Why we need it:
Adds chat message history to the GenAI Request. This will make for better responses from VertexAI.

Adds the request type NPC which allows for making requests to the Google for Games GenAI NPC API.

Which issue(s) this PR fixes:
N/A

Special notes for your reviewer:
This works with Vertex, so in https://github.com/googleforgames/GenAI-quickstart/blob/205ba78aa65782f00a8654e31c88efdc4fdaded6/genai/api/npc_chat_api/config/config.toml#L20-L21 comment out "GKEGenAI", and uncomment "VertexAI".

You will need to first start up and run the GenAI Quickstart cluster, then start and run the NPC chat server.

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: c349792d-03aa-423f-9020-4f57983d9f83

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

  • git fetch https://github.com/googleforgames/agones.git pull/3679/head:pr_3679 && git checkout pr_3679
  • helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.39.0-dev-243e132-amd64

examples/simple-genai-server/gameserver_npcchat.yaml Outdated Show resolved Hide resolved
examples/simple-genai-server/gameserver_npcchat.yaml Outdated Show resolved Hide resolved
examples/simple-genai-server/main.go Outdated Show resolved Hide resolved
Separates out the NPC Request from the GenAIRequest. Updates the NPC Request to include the now required FromID and ToID fields.
@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 42a5224e-c7a7-4a1b-979a-ff3853382f25

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

  • git fetch https://github.com/googleforgames/agones.git pull/3679/head:pr_3679 && git checkout pr_3679
  • helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.39.0-dev-dc7b706-amd64

@zmerlynn zmerlynn enabled auto-merge (squash) March 8, 2024 18:37
@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 46f40a08-5f1f-4757-93a7-00e899ebe545

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 3889e6d1-4214-4267-a38d-ecee6a421460

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

  • git fetch https://github.com/googleforgames/agones.git pull/3679/head:pr_3679 && git checkout pr_3679
  • helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.39.0-dev-70f76f5-amd64

@zmerlynn zmerlynn merged commit 7beb4d2 into googleforgames:main Mar 8, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/documentation Documentation for Agones size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants