-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[LLM pipeline] Add filter out short texts component #247
[LLM pipeline] Add filter out short texts component #247
Conversation
Co-authored-by: NielsRogge <48327001+NielsRogge@users.noreply.github.com>
Co-authored-by: NielsRogge <48327001+NielsRogge@users.noreply.github.com>
Co-authored-by: Robbe Sneyders <robbe.sneyders@gmail.com>
Co-authored-by: NielsRogge <48327001+NielsRogge@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @mrchtr!
Looks like you'll need to update this branch, as there's currently some conflicts on code that I don't think should be included in this PR (the language_filter
component).
Apart from that and some comments to update the component structure to the latest version, it looks good to me.
This component filter out short text passages. Text length can be passed as argument. This component is needed for the LLM dataset creation pipeline. --------- Co-authored-by: NielsRogge <48327001+NielsRogge@users.noreply.github.com> Co-authored-by: Robbe Sneyders <robbe.sneyders@gmail.com>
This component filter out short text passages. Text length can be passed as argument.
This component is needed for the LLM dataset creation pipeline.