JuliaManifolds / ManifoldsBase.jl

Basic interface for manifolds in Julia
https://juliamanifolds.github.io/ManifoldsBase.jl/
MIT License
83 stars 8 forks source link

Fix allocation issue in ManifoldDiff.jl #152

Closed mateuszbaran closed 1 year ago

mateuszbaran commented 1 year ago

This fixes the issue discovered here: https://github.com/JuliaManifolds/ManifoldDiff.jl/pull/18 .

codecov[bot] commented 1 year ago

Codecov Report

Merging #152 (6cc0c43) into master (f3f906d) will decrease coverage by 0.12%. The diff coverage is 100.00%.

:exclamation: Current head 6cc0c43 differs from pull request most recent head 3da980a. Consider uploading reports for the commit 3da980a to get more accurate results

@@            Coverage Diff             @@
##           master     #152      +/-   ##
==========================================
- Coverage   99.96%   99.84%   -0.12%     
==========================================
  Files          19       19              
  Lines        2529     2529              
==========================================
- Hits         2528     2525       -3     
- Misses          1        4       +3     
Impacted Files Coverage Δ
src/exp_log_geo.jl 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more