Skip to content

Container enhancements #142

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.

Sign up for GitHub

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

Jump to bottom
Open
coleramos425 wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Container enhancements #142

coleramos425 wants to merge 3 commits into from

Conversation

@coleramos425
Copy link
Collaborator

@coleramos425 coleramos425 commented Aug 18, 2025

This PR merges apptainer/ and docker/ into a unified container/ subdir simplifying user experience with a single shell script, i.e.,

# To use docker
./containers/run.sh --docker
# To use apptainer
./containers/run.sh --apptainer

coleramos425 and others added 3 commits August 18, 2025 17:35
@coleramos425
Streamline containerization into a single subdir and run.sh script
bfbaaf2 
Signed-off-by: coleramos425 <colramos@amd.com>
@coleramos425
Ruff formatting
495e981 
Signed-off-by: coleramos425 <colramos@amd.com>
@github-actions
Apply Ruff auto-fixes
5d4161e
@coleramos425 coleramos425 mentioned this pull request Aug 18, 2025
Open
@mawad-amd mawad-amd force-pushed the colramos/container-updates branch from 3288a95 to 5d4161e Compare November 8, 2025 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mawad-amd mawad-amd Awaiting requested review from mawad-amd mawad-amd is a code owner

@JoseSantosAMD JoseSantosAMD Awaiting requested review from JoseSantosAMD

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@coleramos425