Skip to content

Conversation

@sjkelly
Copy link
Contributor

@sjkelly sjkelly commented Jan 31, 2023

This should help some instances where things want to precompile code. From my benchmarks there is a de minimis reduction in load time.

This should help some instances where things want to
precompile code. From my benchmarks there is a de minimis
reduction in load time.
@sjkelly sjkelly marked this pull request as ready for review January 31, 2023 17:13
@codecov
Copy link

codecov bot commented Jan 31, 2023

Codecov Report

Base: 72.23% // Head: 71.92% // Decreases project coverage by -0.31% ⚠️

Coverage data is based on head (875cc6e) compared to base (0b037f2).
Patch coverage: 29.16% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #245      +/-   ##
==========================================
- Coverage   72.23%   71.92%   -0.31%     
==========================================
  Files          23       23              
  Lines         940      944       +4     
==========================================
  Hits          679      679              
- Misses        261      265       +4     
Impacted Files Coverage Δ
ext/DiffEqBiologicalExt.jl 0.00% <0.00%> (ø)
ext/ParameterizedFunctionsExt.jl 0.00% <0.00%> (ø)
src/Latexify.jl 35.29% <0.00%> (-10.86%) ⬇️
ext/SymEngineExt.jl 80.00% <50.00%> (ø)
ext/DataFramesExt.jl 93.33% <85.71%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@sjkelly sjkelly changed the title WIP: switch to Weak dependencies on 1.9+ Switch to Weak dependencies on 1.9+ Feb 2, 2023
@gustaphe gustaphe merged commit a9cc166 into korsbo:master Apr 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants