Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix build with sundials 7 #94

Merged
merged 9 commits into from
May 17, 2024

Conversation

jschueller
Copy link
Contributor

No description provided.

@jschueller jschueller force-pushed the core2 branch 8 times, most recently from c6beb0d to a76687d Compare May 15, 2024 09:26
@jschueller jschueller changed the title Fix linking to sundials 7 Fix build with sundials 7 May 15, 2024
@jschueller jschueller force-pushed the core2 branch 3 times, most recently from 086fd58 to 30ffcf5 Compare May 15, 2024 15:19
@jschueller
Copy link
Contributor Author

jschueller commented May 15, 2024

Could you review this one @modelonrobinandersson / @PeterMeisrimelModelon ?
It fixes several issues for building with sundials 7.x
Also fixes the cvode NULL pointer bug shown by the tests and the nfcnjacs number test (see IDAGetNumLinResEvals)

@jschueller
Copy link
Contributor Author

jschueller commented May 15, 2024

would it be possible to have a new release after that ? it would ease packaging for conda etc

Copy link
Member

@modelonrobinandersson modelonrobinandersson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jschueller I think we are just missing a changelog entry, other than that it looks good to me.

@jschueller
Copy link
Contributor Author

@modelonrobinandersson done, and added one last commit to fix latex formulas escape (avoids SyntaxWarning)

@modelonrobinandersson
Copy link
Member

@modelonrobinandersson done, and added one last commit to fix latex formulas escape (avoids SyntaxWarning)

looks good to me!

@jschueller
Copy link
Contributor Author

good for you too @PeterMeisrimelModelon ?

@PeterMeisrimelModelon PeterMeisrimelModelon merged commit 214dbd6 into modelon-community:master May 17, 2024
2 checks passed
PeterMeisrimelModelon pushed a commit that referenced this pull request May 17, 2024
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.

3 participants