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

Apptainer/Singularity containers #140

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

SouzaEM
Copy link
Collaborator

@SouzaEM SouzaEM commented Jan 23, 2025

Closes #138

@SouzaEM SouzaEM requested a review from Olender January 23, 2025 13:27
@SouzaEM SouzaEM self-assigned this Jan 23, 2025
@SouzaEM SouzaEM linked an issue Jan 23, 2025 that may be closed by this pull request
3 tasks
@Olender
Copy link
Collaborator

Olender commented Jan 24, 2025

Is our CI runner down again? I will see if I can get it running to test this

Copy link
Collaborator

Choose a reason for hiding this comment

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

Are we incompatible with newer meshio?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I built a image without specifying a version of meshio and all tests run successfully. Actually, I don't remember anymore why I fixed meshio version. It may be related to other implementations I was doing. I will roll back this change.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I specially like that there are instructions for using it in mintrop! Does it work well in parallel?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes. I did not put the parallel test in the example because it fails when not running from spyro root directory. I still did not figure out why.

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.

Apptainer/Singularity container
2 participants