Skip to content

Conversation

@tzolov
Copy link
Contributor

@tzolov tzolov commented Aug 29, 2024

No description provided.

@tzolov tzolov marked this pull request as ready for review September 3, 2024 15:04
@tzolov tzolov changed the title [WIP] Handling stream advisor responses Handling stream advisor responses Sep 3, 2024
@markpollack markpollack self-assigned this Sep 3, 2024

}

public static Prompt toPrompt(DefaultChatClientRequestSpec advisedRequest, String formatParam) {
Copy link
Member

Choose a reason for hiding this comment

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

could be protected instead of public.

advisedResponse = advisor.adviseResponse(advisedResponse, context);
advisedResponse = AdvisorObservableHelper.adviseResponse(parentObservation, advisor,
advisedResponse, context);
// advisedResponse = advisor.adviseResponse(advisedResponse, context);
Copy link
Member

Choose a reason for hiding this comment

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

removed commented out line


@Override
public StreamResponseMode getStreamResponseMode() {
return StreamResponseMode.ON_FINISH_REASON;
Copy link
Member

Choose a reason for hiding this comment

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

why not AGGREGATE?

@tzolov
Copy link
Contributor Author

tzolov commented Sep 10, 2024

Merged at bf8dabf

@tzolov tzolov closed this Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants