Skip to content

Review tutorial: Intro to HPC - #188

Merged
s-sajid-ali merged 12 commits into
mainfrom
review_tutorial_intro_hpc
Oct 15, 2025
Merged

Review tutorial: Intro to HPC#188
s-sajid-ali merged 12 commits into
mainfrom
review_tutorial_intro_hpc

Conversation

@RobJY

@RobJY RobJY commented Oct 7, 2025

Copy link
Copy Markdown
Contributor

Mainly minor changes, but this is new, so please let me know if you have any suggestions or see anything that should be changed. Thanks!

A couple of questions:

  • Should we make our own version of the tutorial `HPC Parallelisation For Novices? (we link to it in 08_running_parallel_job.mdx)
  • Should most of the Transferring Data section of 10_using_resources_responsibly.mdx be moved to 07_transferring_files_remote.mdx?

@RobJY
RobJY requested review from mdweisner and s-sajid-ali October 7, 2025 15:32
@RobJY RobJY self-assigned this Oct 7, 2025
@github-actions

github-actions Bot commented Oct 7, 2025

Copy link
Copy Markdown
Contributor
PR Preview Action v1.6.2
Preview removed because the pull request was closed.
2025-10-15 15:14 UTC

@s-sajid-ali

s-sajid-ali commented Oct 7, 2025

Copy link
Copy Markdown
Member

Should we make our own version of the tutorial `HPC Parallelisation For Novices? (we link to it in 08_running_parallel_job.mdx)

Can do, to me it's not the highest priority task.

Should most of the Transferring Data section of 10_using_resources_responsibly.mdx be moved to 07_transferring_files_remote.mdx?

Definitely!

Thanks for catching and fixing so many minor issues!

@RobJY

RobJY commented Oct 7, 2025

Copy link
Copy Markdown
Contributor Author

Should we make our own version of the tutorial `HPC Parallelisation For Novices? (we link to it in 08_running_parallel_job.mdx)

Can do, to me it's not the highest priority task.

Sounds good. I'll go back to working on ML/AI then and make a ticket for this.

Should most of the Transferring Data section of 10_using_resources_responsibly.mdx be moved to 07_transferring_files_remote.mdx?

Definitely!

ok, cool, I'll make that change and commit it as part of this PR and we can discuss if needed.

I'm happy to make the updates. I think it gets a little better with each pass. Cheers!

@RobJY

RobJY commented Oct 7, 2025

Copy link
Copy Markdown
Contributor Author

Ok, I've moved that section. Please let me know if you'd like any changes. Thanks!

@s-sajid-ali
s-sajid-ali merged commit f2e6906 into main Oct 15, 2025
5 checks passed
@s-sajid-ali
s-sajid-ali deleted the review_tutorial_intro_hpc branch October 15, 2025 15:13
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