rapidsai/cudf

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

Geschlossen

#19.720 geöffnet am 18.08.2025

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (735 Forks)batch import
Pythonbuggood first issue

Repository-Metriken

Stars
 (6.000 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 17T 21h) (230 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Beginnen Sie mit dem Bau von pylibcudf mit aktivierten Warnungen, um das Problem zu reproduzieren (siehe verlinkte CI Protokolle). Untersuchen Sie die Cython .pyx Dateien in pylibcudf auf Funktionen, die 'except *' oder 'except +' Deklarationen haben, aber C++ Funktionen aufrufen, die als noexcept markiert sind. Vergleichen Sie mit dem Ansatz im RMM PR (rapidsai/rmm/pull/2020), der noexcept zu nicht werfenden C++ Funktionen hinzugefügt hat. Das Ziel ist es, noexcept zu allen solchen C++ Funktionen in cuDF hinzuzufügen, die aus Cython aufgerufen werden, und unnötige except Klauseln in .pyx Dateien zu entfernen.
Tech Stack
cpppython
Domain
dataperformance
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
4
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++ exception handlingCython syntaxBuilding pylibcudf with warnings
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
45