rapidsai/cudf

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

已關閉

#19,720 建立於 2025年8月18日

在 GitHub 查看貢獻者指南
 (3 則留言) (0 個反應) (0 位負責人)C++ (735 個分叉)batch import
Pythonbuggood first issue

倉庫指標

星標
 (6,000 顆星)
PR 合併指標
 (平均合併 17天 21小時) (30 天內合併 230 個 PR)

描述

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

貢獻者指南

研究方向
首先啟用警告建構 pylibcudf 以重現問題（參見連結的 CI 日誌）。檢查 pylibcudf 中的 Cython .pyx 檔案，尋找宣告了 'except *' 或 'except +' 但呼叫了標記為 noexcept 的 C++ 函式的函式。參考 RMM PR（rapidsai/rmm/pull/2020）中的方法，該方法為不拋出異常的 C++ 函式添加了 noexcept。目標是為 cuDF 中所有從 Cython 呼叫的此類 C++ 函式添加 noexcept，並移除 .pyx 檔案中不必要的 except 子句。
技術棧
cpppython
領域
dataperformance
議題類型
錯誤
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
4
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1-2 天
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
新近可參與
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
清晰
前置要求
C++ exception handlingCython syntaxBuilding pylibcudf with warnings
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
45