Skip to content

Conversation

julian-risch
Copy link
Member

@julian-risch julian-risch commented Apr 17, 2025

fixes #394

@bilgeyucel There seems to be an issue with the Serialization/Deserialization that I couldn't investigate further.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link
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.

@julian-risch I have updated the tutorial (extended the conclusion, changed some explanation etc.), I think it's good to go as is.
Let's resolve the serde issue, but it's completely fine if we remove that section from the tutorial.

@julian-risch
Copy link
Member Author

serde issue is a bug in DocumentPreprocessor. I discussed with Sebastian and he opened a Haystack PR: deepset-ai/haystack#9277

@julian-risch
Copy link
Member Author

The bug is fixed now and the tutorial runs without issues.

@julian-risch julian-risch marked this pull request as ready for review April 22, 2025 07:07
@julian-risch julian-risch requested a review from a team as a code owner April 22, 2025 07:07
Copy link
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.

LGTM! ✅
We need to update the created_at date and the pip install line before publishing

@julian-risch julian-risch merged commit 7c7e439 into main Apr 22, 2025
3 of 4 checks passed
@julian-risch julian-risch deleted the supercomponent-tutorial branch April 22, 2025 16:17
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.

Create tutorial for SuperComponent
2 participants