Check for repeated unthunking
まだ誰も着手していません。
評価
調査の方向性
報告された 193 行目付近の失敗について testers.jl の test_rrule から始め、繰り返し unthunking する場合の挙動について ChainRules.jl の pull request 670 を確認してください。複数回 unthunked される thunk と、同じ thunk が複数箇所で返される場合に期待される処理を定義し、その後、選択した挙動を示す回帰テストのカバレッジを追加してください。
索引モデルが issue の本文から書いたものです。
説明
test_rrule checks that the rule accepts thunks.
Should it also check that these aren't accidentally un-thunked more than once, by some passing in some @thunk (COUNT[+=1]; val) thing? That's a mistake although not a wrong-answers one, just slow. As seen in https://github.com/JuliaDiff/ChainRules.jl/pull/670
julia> using ChainRulesCore, ChainRulesTestUtils
julia> plus(x,y) = x+y;
julia> unplus(z) = (NoTangent(), z, z);
julia> unplus(::AbstractThunk) = error("fails");
julia> ChainRulesCore.rrule(::typeof(plus), x, y) = x+y, unplus;
julia> test_rrule(plus, [1.0], [2.0])
test_rrule: plus on Vector{Float64},Vector{Float64}: Error During Test at /Users/me/.julia/packages/ChainRulesTestUtils/YbVdW/src/testers.jl:193
Got exception outside of a @test
fails
julia> unplus(z::AbstractThunk) = (NoTangent(), unthunk(@show z), @show unthunk(z))
julia> test_rrule(plus, [1.0], [2.0]);
z = Thunk(ChainRulesTestUtils.var"#61#65"{Vector{Float64}}([1.73]))
unthunk(z) = [1.73]
z = Thunk(ChainRulesTestUtils.var"#62#66"{Vector{Float64}}([1.73]))
unthunk(z) = [1.73]
Test Summary: | Pass Total Time
test_rrule: plus on Vector{Float64},Vector{Float64} | 9 9 0.0s
Going further, should it also reject (or warn about) unplus(z::AbstractThunk) = (NoTangent(), z, z);? If what's returned contains exactly the same thunk it got, in more than one place, then it's likely that further steps will unthunk twice.
- 主要言語
- Julia
- スター
- 50
- フォーク
- 18
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
JuliaDiff/ChainRulesTestUtils.jl のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
JuliaDiff/ChainRulesTestUtils.jl#148 · リアクション 1 件 ·
-
use DocThemeIndigoオープンdocumentation good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 45/100
-
mutation
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
JuliaDiff/ChainRulesTestUtils.jl の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
SciML/OrdinaryDiffEq.jl#4875 ·
メンテナーはふだん 3 日以内に返信
-
documentation
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
SciML/NonlinearSolve.jl#1372 ·
メンテナーはふだん 1 日以内に返信
-
Truncated discrete ccdf and logccdf return incorrect values at the lower atom対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
JuliaStats/Distributions.jl#2095 ·
-
documentation 📜 microphysics💧 ❄️
難易度 2/5 1〜3時間 初心者へのやさしさ 63/100
NumericalEarth/Breeze.jl#1073 ·
メンテナーはふだん 1 日以内に返信