rapidsai/cudf

Audit pylibcudf for `noexcept`/`nogil` and remove "Exception check … requires the GIL" warnings

Fechada

#19.720 aberto em 18 de ago. de 2025

Ver no GitHubGuia do colaborador
 (3 comentários) (0 reação) (0 responsável)C++ (735 forks)batch import
Pythonbuggood first issue

Métricas do repositório

Stars
 (6.000 estrelas)
Métricas de merge de PR
 (Mesclagem média 17d 21h) (230 fundiu PRs em 30d)

Description

Describe the bug When building pylibcudf, our CI logs show many performance warnings like:

performance hint: /__w/cudf/cudf/python/pylibcudf/pylibcudf/labeling.pyx:56:22:
Exception check after calling 'view' will always require the GIL to be acquired.
Declare 'view' as 'noexcept' if you control the definition and you're sure you
don't want the function to raise exceptions.

Example

https://github.com/rapidsai/cudf/actions/runs/17040978623/job/48305629162?pr=19446#step:12:330

Steps/Code to reproduce bug build pylibcudf with warnings

We should check pylibcudf for places where we’ve declared except * or except + but the C++ is non-throwing.

Expected behavior No perf warnings

Additional context Was recently fixed in RMM. See https://github.com/rapidsai/rmm/pull/2020

Guia do colaborador

Direção de pesquisa
Comece construindo o pylibcudf com avisos habilitados para reproduzir o problema (veja os logs do CI vinculados). Examine os arquivos .pyx do Cython no pylibcudf em busca de funções que tenham declarações 'except *' ou 'except +', mas que chamem funções C++ marcadas como noexcept. Compare com a abordagem no PR do RMM (rapidsai/rmm/pull/2020) que adicionou noexcept a funções C++ que não lançam exceções. O objetivo é adicionar noexcept a todas essas funções C++ no cuDF chamadas a partir do Cython e remover cláusulas except desnecessárias nos arquivos .pyx.
Pilha de tecnologia
cpppython
Domain
dataperformance
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
4
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++ exception handlingCython syntaxBuilding pylibcudf with warnings
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
45