Skip to content

Raise minimum sympy version to 1.7.1#1368

Merged
FFroehlich merged 2 commits intodevelopfrom
fix_1367_sympy
Dec 17, 2020
Merged

Raise minimum sympy version to 1.7.1#1368
FFroehlich merged 2 commits intodevelopfrom
fix_1367_sympy

Conversation

@dweindl
Copy link
Member

@dweindl dweindl commented Dec 17, 2020

Closes #1367

#1356 tried to remain compatible to sympy 1.6.* and 1.7.*, but failed for _CXXCodePrinterBase (sympy.printing.cxx vs. sympy.printing.cxxcode). I'd prefer to just raise the requirement instead of trying both modules.

@codecov
Copy link

codecov bot commented Dec 17, 2020

Codecov Report

Merging #1368 (70d8f19) into develop (326fe53) will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1368      +/-   ##
===========================================
+ Coverage    78.74%   78.76%   +0.02%     
===========================================
  Files           63       63              
  Lines         9933     9933              
===========================================
+ Hits          7822     7824       +2     
+ Misses        2111     2109       -2     
Flag Coverage Δ
cpp 75.31% <ø> (+0.02%) ⬆️
petab 69.51% <ø> (ø)
python 68.09% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/sundials_matrix_wrapper.cpp 83.82% <0.00%> (+0.42%) ⬆️

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@FFroehlich FFroehlich merged commit 72e70cf into develop Dec 17, 2020
@dweindl dweindl deleted the fix_1367_sympy branch January 28, 2021 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants