search

TS-CUBED / CirculatorySystemModels.jl

MIT License
16 stars 5 forks source link

Reduce compile time #31

Closed termi-official closed 2 months ago

termi-official commented 3 months ago

DifferentialEquations.jl pulls in more solvers than the package needs. Also the package only uses the solvers in the tests. So this PR removes the dependency on DifferentialEquations.jl and adds OrdinaryDiffEq.jl as test dependency.

codecov-commenter commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 43.75%. Comparing base (3095ac1) to head (b0aecc8).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #31 +/- ## ======================================= Coverage 43.75% 43.75% ======================================= Files 1 1 Lines 144 144 ======================================= Hits 63 63 Misses 81 81 ```

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

TS-CUBED commented 2 months ago

Thank you very much for this improvement. Much appreciated.