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