JuliaDiff / ReverseDiff.jl

Reverse Mode Automatic Differentiation for Julia
Other
348 stars 57 forks source link

Fix tape printing #252

Closed gdalle closed 6 months ago

gdalle commented 6 months ago

Fixes https://github.com/EnzymeAD/Enzyme.jl/issues/1354?notification_referrer_id=NT_kwDOAVvVTrM5OTI3ODA3ODE2OjIyNzk1NTk4#issuecomment-2053902656

codecov[bot] commented 6 months ago

Codecov Report

Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 81.47%. Comparing base (c982cde) to head (d1fc4b6). Report is 1 commits behind head on master.

:exclamation: Current head d1fc4b6 differs from pull request most recent head f5ce5fb. Consider uploading reports for the commit f5ce5fb to get more accurate results

Files Patch % Lines
src/tape.jl 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #252 +/- ## ========================================== - Coverage 81.53% 81.47% -0.07% ========================================== Files 18 18 Lines 1587 1576 -11 ========================================== - Hits 1294 1284 -10 + Misses 293 292 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

gdalle commented 6 months ago

Can you add a test (eg using string or an IOBuffer?)?

done