Skip to content

HYPRE: add hypre initialize function (to fix crash)#267

Open
StevenvdLinden wants to merge 1 commit intodalesteam:devfrom
StevenvdLinden:dev-hyprefix
Open

HYPRE: add hypre initialize function (to fix crash)#267
StevenvdLinden wants to merge 1 commit intodalesteam:devfrom
StevenvdLinden:dev-hyprefix

Conversation

@StevenvdLinden
Copy link

Added explicit call to the HYPRE initialize function. Without this (unexpected) crashes may occur.

@StevenvdLinden StevenvdLinden requested a review from fjansson March 2, 2026 09:04
@StevenvdLinden
Copy link
Author

Fails automated hypre test in which libhypre-dev is automatically installed. In my testbuilds, I built latest Hypre release locally on machine

@fjansson
Copy link
Contributor

fjansson commented Mar 2, 2026

Possibly HYPRE_Initialize() is only present in the new version. There is a HYPRE_Init() "for backwards compatibility". Perhaps that one is available in both the old and new versions? I'd rather not make DALES incompatible with HYPRE 2.x just yet, since that's what you get in fairly recent ubuntu for example.
We could do some #ifdef logic, but that feels like a lot of variants to maintain for something we don't use a lot.

@fjansson
Copy link
Contributor

Replacing HYPRE_Initialize by HYPRE_Init makes it compile on Ubuntu 22 with HYPRE 2.28. @StevenvdLinden can you do that change and try on your machine with the new hypre?

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