Skip to content

add a new apptainer tutorial secton#366

Merged
s-sajid-ali merged 37 commits into
mainfrom
Amanda-dong
Jun 29, 2026
Merged

add a new apptainer tutorial secton#366
s-sajid-ali merged 37 commits into
mainfrom
Amanda-dong

Conversation

@Amanda-dong

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Jun 2, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-06-29 18:19 UTC

Removed unused functions for HTML stripping and MDX building.
@s-sajid-ali

Copy link
Copy Markdown
Member

Instead of adding this at docs/hpc/13_tutorial_intro_hpc/11_apptainer_on_torch.mdx, you can create a new section at docs/hpc/14_tutorial_apptainer

Additionally, since you used this as the source: carpentries-incubator.github.io/apptainer-introduction/index.html , you can have multiple pages as the carpentries tutorial has. This is a tutorial intended for someone who has never used apptainer before (or needs a refresher on some concepts), so condensing this makes it less usable for newcomers.

@s-sajid-ali s-sajid-ali left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@@ -0,0 +1,67 @@
## Why Customize a Container Environment?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

First heading needs to be at # level rather than at ## level.

@s-sajid-ali s-sajid-ali changed the title Amanda dong add a new apptainer tutorial secton Jun 24, 2026
@s-sajid-ali s-sajid-ali requested a review from RobJY June 24, 2026 21:26

@RobJY RobJY left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks great.

I made a few suggestions, but nothing required.

Comment thread docs/hpc/14_tutorial_apptainer/01_intro_apptainer.mdx Outdated
Comment thread docs/hpc/14_tutorial_apptainer/02_running_containers.mdx Outdated
Comment thread docs/hpc/14_tutorial_apptainer/02_running_containers.mdx Outdated
Comment thread docs/hpc/14_tutorial_apptainer/03_apptainer_files.mdx
Comment thread docs/hpc/14_tutorial_apptainer/03_apptainer_files.mdx Outdated
Amanda-dong and others added 10 commits June 28, 2026 23:30
Co-authored-by: Robert Young <robert.jay.young@gmail.com>
Co-authored-by: Robert Young <robert.jay.young@gmail.com>
Co-authored-by: Robert Young <robert.jay.young@gmail.com>
Co-authored-by: Robert Young <robert.jay.young@gmail.com>
Co-authored-by: Robert Young <robert.jay.young@gmail.com>
…ents.mdx

Co-authored-by: Robert Young <robert.jay.young@gmail.com>
@s-sajid-ali s-sajid-ali merged commit 0723507 into main Jun 29, 2026
5 checks passed
@s-sajid-ali s-sajid-ali deleted the Amanda-dong branch June 29, 2026 18:18
@s-sajid-ali s-sajid-ali mentioned this pull request Jun 29, 2026
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.

3 participants