Removing / with tangent denominator

未关闭
  • 哪里不对？
#665 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
3/5
预计耗时
1-2 天
新手友好度
35/100
Issue 类型
重构
描述清晰度
基本清楚
活跃度
停滞
技术栈
julia
领域
backend-api-design

调研方向

检查 issue 中所示的除法重载，并对比 #643 中相关的更改。找出 AbstractThunk 的其他除法方法，保留 tangent-by-scalar 缩放，并移除此处描述的 scalar-by-tangent 行为；添加或更新测试，以确认预期的除法情况。

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

描述

breaking

along the same lines as #643
We should not have e.g.

Base.:(/)(a, b::AbstractThunk) = a / unthunk(b) 
Base.:(/)(a::AbstractThunk, b::AbstractThunk) = unthunk(a) / unthunk(b)

We have a few of these

Since while it makes sense to divide a tangent by a scalar (its just scaling down) it does done make sense to divide a scalar by a tangent.

主要语言
Julia
星标
267
派生
66
平均合并
1 天 1 小时
30 天内合并 PR
9

环境准备

这个项目没有提供开发容器、Dockerfile 或贡献指南，环境需要你自己搭建：先看它的 README，通用步骤见我们的新手贡献指南。

从这里开始

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

JuliaDiff/ChainRulesCore.jl 的其他 Issue

查看 JuliaDiff/ChainRulesCore.jl 的全部 Issue

相似的 Issue

更多 Julia Issue

把新 issue 发到你的邮箱

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