-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Simplify tutorial and README #216
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #216 +/- ##
=======================================
Coverage 95.60% 95.60%
=======================================
Files 73 73
Lines 3820 3820
=======================================
Hits 3652 3652
Misses 168 168 ☔ View full report in Codecov by Sentry. |
I will make some more changes to this once #217 is merged. |
it is merged, you can track all the places in the docs where ADTypes is used and remove them |
Ready for review! |
@gdalle A test on coloring currently fails: Coloring: Test Failed at /home/runner/work/DifferentiationInterface.jl/DifferentiationInterface.jl/DifferentiationInterface/test/coloring.jl:11
Expression: maximum(row_colors) < size(A, 1) ÷ 2
Evaluated: 51 < 50
Stacktrace:
[1] macro expansion
@ /opt/hostedtoolcache/julia/1.10.2/x64/share/julia/stdlib/v1.10/Test/src/Test.jl:672 [inlined]
[2] top-level scope
@ ~/work/DifferentiationInterface.jl/DifferentiationInterface.jl/DifferentiationInterface/test/coloring.jl:508 |
It's a randomized test, should nearly never fail. Fix the seed and call it a day for now |
@repl
blocks by@example
blocks to reduce visual noise (fewerjulia>
,;
and newlines)using ADTypes
instead ofimport ADTypes