QuantumSavory / QuantumSymbolics.jl

Computer algebra tools for symbolic manipulations in quantum mechanics and quantum information
MIT License
29 stars 9 forks source link

Updates according to latest SymbolicUtils and TermInterface versions #42

Closed apkille closed 2 months ago

apkille commented 2 months ago

Merged the CompatHelpers for SymbolicUtils (#37) and TermInterface (#39). Made changes according to latest docs: https://juliasymbolics.github.io/TermInterface.jl/dev/.

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 30.00000% with 42 lines in your changes missing coverage. Please review.

Project coverage is 54.70%. Comparing base (ecd9f15) to head (32cf192). Report is 2 commits behind head on main.

Files Patch % Lines
src/QSymbolicsBase/basic_ops_inhomogeneous.jl 10.00% 18 Missing :warning:
src/QSymbolicsBase/basic_ops_homogeneous.jl 28.57% 10 Missing :warning:
src/QSymbolicsBase/predefined.jl 40.00% 9 Missing :warning:
src/QSymbolicsBase/latexify.jl 0.00% 4 Missing :warning:
ext/QuantumCliffordExt/QuantumCliffordExt.jl 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #42 +/- ## ========================================== - Coverage 58.45% 54.70% -3.75% ========================================== Files 14 14 Lines 426 446 +20 ========================================== - Hits 249 244 -5 - Misses 177 202 +25 ```

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

Krastanov commented 2 months ago

This is wonderful, thank you! I made a small change to hopefully fix the Downgrade checks (they basically check whether the oldest permitted versions of dependencies still work -- in this case they should not, given that isexpr did not exist).

Krastanov commented 2 months ago

just a quick explanation for what I am doing with the Downgrade CI runs and why so many compat bound changes are necessary.

There are two things that Downgrade checks for:

Most software projects do not bother with this type of Downgrade check in their CI, but it provides for less confusion and installation issues for users, so I find it worthwhile.