Add Prior Labs agent cookbook#281
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
| @@ -0,0 +1,482 @@ | |||
| { | |||
There was a problem hiding this comment.
Line #13. # We need to rewrite the previous assistant replies to user
Maybe a better explanation is "this particular provider expects the conversation to start with a user message".
Reply via ReviewNB
There was a problem hiding this comment.
I'm confused here. Don't all conversations end with an assistant message?
There was a problem hiding this comment.
Yes, they do! In our case, the last message from the "summarizer" is an assistant message (a Markdown table with the hotel details), so the conversation ends with an assistant message when we pass it to the agent (that's what Anthropic does not support - the last message has to be a user message with an instruction on what to do next).
But I feel I got the confusion here. A cleaner approach might be to just add a simple user message to the sequence and avoid having this prompt builder. Let me try improve it.
| @@ -0,0 +1,482 @@ | |||
| { | |||
There was a problem hiding this comment.
More of a detailed ask: can you display the content in markdown format, but just the table, for comparison? Splitting the string from "| Hotel Name|" could be beneficial here.
Reply via ReviewNB
bilgeyucel
left a comment
There was a problem hiding this comment.
Thanks @kacperlukawski, I love the use case! I left some comments, mostly around the formatting.
Additionally, can you add the notebook details in index.toml, otherwise, it wont be listed on the website
|
@bilgeyucel I would appreciate another look at the cookbook |
bilgeyucel
left a comment
There was a problem hiding this comment.
I only added one extra comment to one thread. Other than that, looks very good!
Adds a new cookbook recipe demonstrating how to combine LLMs and classical tabular ML to enrich scraped web data.
The notebook builds a Haystack pipeline that: