argilla
argilla copied to clipboard
[FEATURE] Update fields of RemoteFeedbackRecords
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
🤖
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 RemoteFeedbackRecord
s 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
Hi @plaguss ! Is this issue related to a specific problem? What do you mean by updating fields?
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
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.
This issue is stale because it has been open for 90 days with no activity.
This issue was closed because it has been inactive for 30 days since being marked as stale.