Skip to content
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

[FEATURE] add text-descriptives support to the to_argilla-methods #231

Closed
davidberenstein1957 opened this issue Jan 10, 2024 · 2 comments · Fixed by #271
Closed

[FEATURE] add text-descriptives support to the to_argilla-methods #231

davidberenstein1957 opened this issue Jan 10, 2024 · 2 comments · Fixed by #271
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@davidberenstein1957
Copy link
Member

Is your feature request related to a problem? Please describe.
We ideally want to enrich data as much as possible when pushing it to Argilla to allow for out-of-the-box support for as many search, filter and sort features as possible.

Describe the solution you'd like
We want to re-use the TextDescriptivesExtractor introduced as Argilla integration through this PR.

dataset = ...
dataset.to_argilla(text_descriptives=True)
if text_descriptives:
    try:
        from argilla import TextDescriptivesExtractor
    except:
         # handle warning and skip
    if isinstance(text_descriptives, TextDescriptivesExtractor):
        td = text_descriptives
    else:
        td = TextDescriptivesExtractor
    td = st.update_dataset(ds)

Describe alternatives you've considered
N.A.

Additional context
N.A.

@davidberenstein1957 davidberenstein1957 added the enhancement New feature or request label Jan 10, 2024
@davidberenstein1957 davidberenstein1957 modified the milestone: 0.4.0 Jan 10, 2024
@davidberenstein1957 davidberenstein1957 self-assigned this Jan 15, 2024
@davidberenstein1957
Copy link
Member Author

default=True

@davidberenstein1957
Copy link
Member Author

We decided to move this issue to 0.5.0 along with #230 because we wanted to outline the fields a bit more which we would like to include by default because Argilla only allows 5 vector settings per record i.e. we can only select 5 fields. And we would like to align both.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant