Hacktoberfest 2026:维护者为十月标记出来的 issue,仍然开放、适合新手。 浏览 Hacktoberfest issue

Duplicate and incomplete definitions of `_graph_nodes` and `grad_check`

未关闭 适合新手
#8 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
65/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
冷清
技术栈
numpy, python

调研方向

首先阅读 leanpass/tensor.py,比较 _graph_nodes 和 grad_check 的两个定义;检查文档中声明的返回类型和现有测试。在删除重复项并完成梯度检查后运行相关测试。完成意味着每个函数仅保留一个正确的定义,且 grad_check 返回文档中声明的错误元组。

由索引模型根据 Issue 内容生成。

描述

good first issue

File: leanpass/tensor.py

The file defines _graph_nodes and grad_check twice. The second definitions overwrite the first ones, but the second grad_check is truncated (numeric[node] = ) and contains several bugs (calls node.zero_grad_all(), uses undefined self.zero_grad_all(), mismatched return types). This makes the gradient‑checking utility unusable and may silently break other code that expects the original implementation.

Fix: Remove the duplicated definitions. Keep a single, correct implementation of _graph_nodes (the first one is fine) and a complete, tested grad_check that:

  • Uses self.zero_grad_all() to clear grads.
  • Computes numerical gradients correctly (e.g., using np.sum(plus) - np.sum(minus) for scalar outputs).
  • Returns a list of error tuples as documented.

File: leanpass/tensor.py

Filed automatically by ai-issue-scan.

主要语言
Python
星标
16
派生
1
平均合并
2 天 8 小时
30 天内合并 PR
5

环境准备

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

Terminay/leanpass 的其他 Issue

查看 Terminay/leanpass 的全部 Issue

相似的 Issue

更多 Python Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。