rapidsai/cudf

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

Fermée

#19 720 ouverte le 18 août 2025

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (0 personne assignée)C++ (735 forks)batch import
Pythonbuggood first issue

Métriques du dépôt

Stars
 (6 000 étoiles)
Métriques de merge PR
 (Merge moyen 17j 21h) (230 PRs mergées en 30 j)

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

Guide contributeur

Direction de recherche
Commencez par construire pylibcudf avec les avertissements activés pour reproduire le problème (voir les journaux CI liés). Examinez les fichiers .pyx Cython dans pylibcudf pour les fonctions qui ont des déclarations 'except *' ou 'except +' mais appellent des fonctions C++ marquées noexcept. Comparez avec l'approche dans la PR RMM (rapidsai/rmm/pull/2020) qui a ajouté noexcept aux fonctions C++ ne lançant pas d'exceptions. L'objectif est d'ajouter noexcept à toutes ces fonctions C++ dans cuDF appelées depuis Cython, et de supprimer les clauses except inutiles dans les fichiers .pyx.
Stack technique
cpppython
Domaine
dataperformance
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++ exception handlingCython syntaxBuilding pylibcudf with warnings
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
45