Remove support for missing response status

[frascuchon]

See the CHANGELOG.md entry line: https://github.com/argilla-io/argilla/blob/develop/CHANGELOG.md?plain=1#L59

[dosubot]

🤖

Hey there, @frascuchon! Good to see you around the Argilla repo again. How's it going?

Based on the information provided in the CHANGELOG.md file, it's clear that the missing response status is deprecated and should be removed in the release v1.24.0. The missing response status is implemented in the ResponseStatusFilter Enum in the src/argilla/server/enums.py file. Here is the relevant code:

class ResponseStatusFilter(str, Enum):
    draft = "draft"
    missing = "missing"  # Deprecated, use pending instead
    pending = "pending"
    submitted = "submitted"
    discarded = "discarded"

Source: src/argilla/server/enums.py

Additionally, there is functionality in the Argilla codebase that relies on the missing response status. The response_statuses parameter in the search_dataset_records function can take a list of ResponseStatusFilter objects, which can include the missing status. This parameter is then passed to the _get_search_responses function, which uses it to filter the search responses.

Here is the relevant code:

@router.post(
    "/datasets/{dataset_id}/records/search",
    status_code=status.HTTP_200_OK,
    response_model=SearchRecordsResult,
    response_model_exclude_unset=True,
)
async def search_dataset_records(
    *,
    db: AsyncSession = Depends(get_async_db),
    search_engine: SearchEngine = Depends(get_search_engine),
    dataset_id: UUID,
    body: SearchRecordsQuery,
    metadata: MetadataQueryParams = Depends(),
    sort_by_query_param: SortByQueryParamParsed,
    include: Optional[RecordIncludeParam] = Depends(parse_record_include_param),
    response_statuses: List[ResponseStatusFilter] = Query([], alias="response_status"),
    offset: int = Query(0, ge=0),
    limit: int = Query(default=LIST_DATASET_RECORDS_LIMIT_DEFAULT, ge=1, le=LIST_DATASET_RECORDS_LIMIT_LE),
    current_user: User = Security(auth.get_current_user),
):
    ...
    search_responses = await _get_search_responses(
        db=db,
        search_engine=search_engine,
        dataset=dataset,
        search_records_query=body,
        limit=limit,
        offset=offset,
        parsed_metadata=metadata.metadata_parsed,
        response_statuses=response_statuses,
        sort_by_query_param=sort_by_query_param,
    )
    ...

Source: src/argilla/server/apis/v1/handlers/datasets/records.py

To remove the missing response status, you would need to remove it from the ResponseStatusFilter Enum and update any functionality that relies on it to use the pending status instead. Please note that as a maintainer, I can't make these changes for you. You would need to make these changes in your local environment and test them thoroughly to ensure that they don't break any existing functionality.

Sources

• CHANGELOG.md
• src/argilla/server/enums.py
• src/argilla/server/apis/v1/handlers/datasets/records.py

---

This response is meant to be useful and save you time. It is not meant to be a precise solution, but rather a starting point for your own research.

Help me be more useful! Please leave a 👍 if this is helpful and 👎 if it is irrelevant.

If you want to continue the conversation, start your reply with @dosu-bot.