argilla icon indicating copy to clipboard operation
argilla copied to clipboard

[FEATURE] Update fields of RemoteFeedbackRecords

Open plaguss opened this issue 1 year ago • 5 comments

Is your feature request related to a problem? Please describe. I would like to update the fields of RemoteFeedbackRecords.

Describe the solution you'd like A clear and concise description of what you want to happen.

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Related to #4328

plaguss avatar Nov 27 '23 16:11 plaguss

🤖

Hello @plaguss,

I'm Dosu, your friendly bot assistant for the Argilla repository! I'm here to help you navigate issues, answer questions, and help you contribute to our project. While we wait for a human maintainer to join us, feel free to ask me anything. I'm here to assist you.

Thank you for your feature request. Based on the information you've provided, it seems you want to update the fields of RemoteFeedbackRecords.

Currently, the update method in the RemoteFeedbackRecords class is used to update a list of RemoteFeedbackRecords in Argilla. The method accepts two arguments: records and show_progress. The records argument can be a single RemoteFeedbackRecord or a list of RemoteFeedbackRecord. The show_progress argument is a boolean that determines whether to show a progress bar while updating the records. Here is the source code for your reference.

As for the fields that can be updated, currently, the RemoteFeedbackRecord class in Argilla allows the suggestions field to be updated. This field can be updated using the update method. It accepts a single SuggestionSchema, a list of SuggestionSchema, a single dictionary, or a list of dictionaries. If a dictionary is provided, it will be converted to a SuggestionSchema internally. Here is the relevant source code.

Regarding your request's relation to issue #4328, without additional context or information about the issue, it's not possible to determine how this method or the proposed changes are related to it. Could you please provide more details about how you would like the fields of RemoteFeedbackRecords to be updated, any alternative solutions or features you've considered, and how this is related to issue #4328?

Also, there was a similar solved issue: allow for updating only certain fields of a record ID via rg.log projection, which was closed by a pull request: feat: Allow partial record update. This might provide some insights into your request.

Looking forward to your response.

Sources

dosubot[bot] avatar Nov 27 '23 16:11 dosubot[bot]

Hi @plaguss ! Is this issue related to a specific problem? What do you mean by updating fields?

nataliaElv avatar Nov 28 '23 10:11 nataliaElv

Hi @nataliaElv! I think this really doesn't make sense, I saw we couldn't update this from the sdk (related to the #4328), but it doesn't make sense. I'll close this

plaguss avatar Nov 28 '23 10:11 plaguss

This is definitely something that has come up in user interviews, but it has some implications in terms of dataset consistency. I'll add this to the triage so we can discuss internally.

nataliaElv avatar Feb 22 '24 13:02 nataliaElv

This issue is stale because it has been open for 90 days with no activity.

github-actions[bot] avatar May 24 '24 01:05 github-actions[bot]

This issue was closed because it has been inactive for 30 days since being marked as stale.

github-actions[bot] avatar Jun 23 '24 01:06 github-actions[bot]