Skip to content

add support for streaming - #10

Open
AngeloGiacco wants to merge 2 commits into
mainfrom
add-text-streaming-support
Open

add support for streaming#10
AngeloGiacco wants to merge 2 commits into
mainfrom
add-text-streaming-support

Conversation

@AngeloGiacco

Copy link
Copy Markdown
Collaborator

No description provided.

@PaulAsjes PaulAsjes left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like the streaming text events are only logged and never exposed to the user via a public API.

AgentChatResponsePartType.START -> {
Log.d("ConvEventHandler", "Agent text stream started")
// Update conversation mode to speaking when streaming starts
_conversationMode.value = ConversationMode.SPEAKING

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit, but SPEAKING might not be totally accurate since technically the agent is writing a message.

Comment thread gradle.properties
# Specifies the JVM arguments used for the daemon process.
# The setting is particularly useful for tweaking memory settings.
org.gradle.jvmargs=-Xmx2048m -Dfile.encoding=UTF-8 --enable-native-access=ALL-UNNAMED --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED
org.gradle.jvmargs=-Xmx2048m -Dfile.encoding=UTF-8 --enable-native-access=ALL-UNNAMED --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.base/java.lang.reflect=ALL-UNNAMED --add-opens=java.base/java.util.concurrent=ALL-UNNAMED

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why this change?

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