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

Add TACC docker image #402

Merged
merged 1 commit into from
Jun 20, 2024
Merged

Conversation

gassmoeller
Copy link
Member

This is a preliminary PR that can create a docker image that runs on TACC Stampede2. So far I have it only running on a single node, and unfortunately the image is quite a bit slower. I suspect this is because I have no access to MKL inside the container. I will think about this a bit more.

@feathern
Copy link
Contributor

Is this still WIP?

@gassmoeller
Copy link
Member Author

Depends how you define WIP. It is not quite done, given the limitations and the fact that the code is not really clean, but I am also not actively working on it at the moment. I hope to come back to it early next year after AGU though. If you want we can close the PR for now and I reopen once I have a better version, but we can also leave it open as a reminder. Either way is fine with me.

@feathern
Copy link
Contributor

OK -- just checking. We can leave it open.

@gassmoeller
Copy link
Member Author

This is in theory ready, but I cannot check right now, because the cluster is closed for scaling tests. Will report back later.

@feathern
Copy link
Contributor

feathern commented May 8, 2024

Checking in again @gassmoeller . Is this ready to merge and/or has it been superseded by any other container work in the past year? Thanks!

@gassmoeller gassmoeller changed the title [WIP] Add TACC docker image Add TACC docker image Jun 18, 2024
@gassmoeller
Copy link
Member Author

This is now ready for review. I have tested that I can build the containers and run them on Stampede3 and Frontera.

Copy link
Contributor

@feathern feathern left a comment

Choose a reason for hiding this comment

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

This looks good. Approved.

@feathern feathern merged commit 8451697 into geodynamics:main Jun 20, 2024
7 checks passed
@gassmoeller gassmoeller deleted the add_tacc_docker_image branch June 20, 2024 15: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