add error message for integers that represent reals
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 38/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- julia
- Área
- testing-qa
Línea de trabajo
Reproduce el fallo mostrado con test_rrule(f, 2) y comienza en el punto de entrada del tester indicado en testers.jl:278. Revisa el caso actual::AbstractFloat y expected::NoTangent sugerido en el issue y, después, verifica que el mensaje de fallo resultante explique claramente la discrepancia de la entrada entera.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
e.g. example by @SomTambe
julia> f(x) = 8x + 10*sin(x)
f (generic function with 1 method)
julia> function ChainRulesCore.rrule(::typeof(f), x)
y = f(x)
function ∇f(Δ)
∇ = 8 + 10*cos(x)
@show ∇ Δ
return ChainRulesCore.NoTangent(), Δ * ∇
end
return y, ∇f
end
julia> test_rrule(f, 2);
∇ = 3.838531634528576
Δ = 2.31
∇ = 3.838531634528576
Δ = 2.31
test_rrule: f on Int64: Test Failed at /home/somvt/.julia/packages/ChainRulesTestUtils/AX7fv/src/testers.jl:278
Expression: ad_cotangent isa NoTangent
Evaluated: 8.86700807576101 isa NoTangent
...
Test Summary: | Pass Fail Total
test_rrule: f on Int64 | 4 1 5
ERROR: Some tests did not pass: 4 passed, 1 failed, 0 errored, 0 broken.
the error message is confusing, we could special case it for actual::AbstractFloat, expected::NoTangent as suggested by @oxinabox
- Lenguaje dominante
- Julia
- Estrellas
- 50
- Forks
- 18
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de JuliaDiff/ChainRulesTestUtils.jl
-
code snippet in readmeAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
JuliaDiff/ChainRulesTestUtils.jl#148 · 1 reacción ·
-
use DocThemeIndigoAbiertodocumentation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
-
more `test_approx` ambiguitiesAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
-
mutation
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
Todos los issues de JuliaDiff/ChainRulesTestUtils.jl
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
JuliaGenAI/HuggingFaceDatasets.jl#96 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
JuliaData/DBFTables.jl#50 ·
-
_fit_phylo_binomial_xlv: default start calls undefined logit (UndefVarError without beta_init)Abiertocode-review
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
JuliaNLSolvers/Optim.jl#1296 ·
-
OptimizationCUTEst jobs fail: gfortran not installed on the runnerPosiblemente ocupada @ChrisRackauckas-Claude la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
SciML/SciMLBenchmarks.jl#1946 · 1 comentario ·
Los mantenedores suelen responder en 1 día