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

conduit static lib #114

Closed
mdavis36 opened this issue Feb 8, 2022 · 3 comments
Closed

conduit static lib #114

mdavis36 opened this issue Feb 8, 2022 · 3 comments
Labels
bug Something isn't working tpls

Comments

@mdavis36
Copy link
Collaborator

mdavis36 commented Feb 8, 2022

When linking conduit as a static lib we see random MPI segfaults in our ats test suite upon exiting each test (even successful runs).

This was seen with conduit v0.7.2 -> v0.8.2.

I'm not certain if conduit is to blame here or Spheral, as we've seen this before in other situations, but this reproducibly triggers it.

@mdavis36 mdavis36 added the bug Something isn't working label Feb 8, 2022
@ptsuji
Copy link
Collaborator

ptsuji commented Feb 8, 2022

Is it necessary to update conduit to 0.8.2 if we update to the changes in your gitlab-ci branch? We're still on 0.5.1.

@mdavis36
Copy link
Collaborator Author

mdavis36 commented Feb 8, 2022

Not technically, but @MishaZakharchanka will be bumping our axom version up soon for the Sidre work which also requires conduit to bump as well.

@mdavis36
Copy link
Collaborator Author

@ptsuji I recently looked into this again. It seems in order to natively support this we will need to bump conduit to 0.8.3, axom to 0.6.1 and include raja per: LLNL/axom#865

We will be pulling raja in as a dependency soon so supporting conduit with a static lib in the Spheral build system is possible. If you don't want to change the conduit and axom versions on your end you probably will be able to link them and there's a good chance things will work, but no promises.

@mdavis36 mdavis36 changed the title conduit static lib & random mpi test failures conduit static lib Jul 18, 2022
@mdavis36 mdavis36 added the tpls label Nov 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working tpls
Projects
None yet
Development

No branches or pull requests

2 participants