Skip to content

Add Prior Labs agent cookbook#281

Merged
kacperlukawski merged 6 commits into
mainfrom
prior-labs-agent
Apr 17, 2026
Merged

Add Prior Labs agent cookbook#281
kacperlukawski merged 6 commits into
mainfrom
prior-labs-agent

Conversation

@kacperlukawski
Copy link
Copy Markdown
Member

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:

  1. Scrapes hotel listings from the web using Firecrawl.
  2. Uses Claude to extract and structure the data into a Markdown table.
  3. Connects to Prior Labs' TabPFN via MCP to predict missing attributes with predicted values marked in bold.

@kacperlukawski kacperlukawski requested a review from a team as a code owner April 1, 2026 16:01
@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Comment thread notebooks/prior_labs_agent.ipynb
Comment thread notebooks/prior_labs_agent.ipynb
Comment thread notebooks/prior_labs_agent.ipynb
Comment thread notebooks/prior_labs_agent.ipynb
@@ -0,0 +1,482 @@
{
Copy link
Copy Markdown
Contributor

@bilgeyucel bilgeyucel Apr 2, 2026

Choose a reason for hiding this comment

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

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

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Changed it

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm confused here. Don't all conversations end with an assistant message?

Copy link
Copy Markdown
Member Author

@kacperlukawski kacperlukawski Apr 15, 2026

Choose a reason for hiding this comment

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

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 @@
{
Copy link
Copy Markdown
Contributor

@bilgeyucel bilgeyucel Apr 2, 2026

Choose a reason for hiding this comment

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

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

Copy link
Copy Markdown
Contributor

@bilgeyucel bilgeyucel left a comment

Choose a reason for hiding this comment

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

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

@kacperlukawski
Copy link
Copy Markdown
Member Author

@bilgeyucel I would appreciate another look at the cookbook

Copy link
Copy Markdown
Contributor

@bilgeyucel bilgeyucel left a comment

Choose a reason for hiding this comment

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

I only added one extra comment to one thread. Other than that, looks very good!

@kacperlukawski kacperlukawski merged commit 3a5bdfd into main Apr 17, 2026
1 check passed
@kacperlukawski kacperlukawski deleted the prior-labs-agent branch April 17, 2026 16:15
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