Piloted and revised Foundational HPC course

[steve-crouch]

A piloted and revised Foundational HPC course based on material from EPCC's Intro to HPC course, comprised of 4 modules.

[martinjrobins]

Thanks @steve-crouch, I can see those link errors arn't coming from your changes, so happy to ignore these! I'll try to get a review done asap

[alasdairwilson]

I am reviewing this in detail but I initially had just a general thought on the layout, I think the high performance computing theme is organised incorrectly.

The "Theme" page has too much specificity in it, take for comparison the technology and tooling theme where one course is on say "IDEs" and another on "Testing" to the HPC theme where one course is on "Parallel Computing" and another is on "Introduction to parallelism" and another on "Introduction to supercomputing". I know this is not all added in this PR though.

There would still need to be work to do to get the rest of the Theme in line and these 4 "streams" could be linked together in the theme diagram too which would not be represented in the default gridbox layout.

For example openmp and MPI would be merged into a single "course" called something like "parallel computing tooling" or "parallel computing libraries" with the openmp and mpi as separate streams.

[alasdairwilson]

Sorry for the mega-review, and for submitting it early by accident, but there you go anyway...

I am happy to leave reorganising the sections for a follow up PR, but I do think the HPC theme needs something done to keep it manageable.

[alasdairwilson]

Apologies @steve-crouch for both the insane amount of suggestions and the fact I accidentally submitted the review part way (meaning some future comments got added as single comments rather than the review)

Anyway, done now 👍

[steve-crouch]

Many thanks for the very comprehensive review, @alasdairwilson! The suggestions were excellent in simplifying and clarifying the text. I've reviewed and (I hope, given the scale of the review comments) accepted all changes. I've also fixed the content's textual references to the prior week-based schedule from the original content (#199) as you suggested.

[alasdairwilson]

So there are 2 markdown lint errors:

These changes should fix them.

[alasdairwilson]

Suggested change

	Supercomputers were first introduced in the 1960s by Seymour Roger Cray at Control Data Corporation (CDC), and have been used intensively in science and engineering ever since.
	Supercomputers were first introduced in the 1960s by Seymour Roger Cray at Control Data Corporation (CDC), and have been used intensively in science and engineering ever since.

[alasdairwilson]

Suggested change

	### Number crunching
	### Number crunching

[alasdairwilson]

Many thanks for the very comprehensive review, @alasdairwilson! The suggestions were excellent in simplifying and clarifying the text. I've reviewed and (I hope, given the scale of the review comments) accepted all changes. I've also fixed the content's textual references to the prior week-based schedule from the original content (#199) as you suggested.

So there are actually like 190 more review comments or something you have to hit the load more to get 20 at a time (or go through the whole review on the changes tab but that's probably more of a pain).

that goes for both the reviews:

#187 (review)

and

#187 (review)

which has 9 or so.

If you are willing to accept them all then you can just "add them to batch" one at a time rather than commiting them directly and then commit when finished, which is a bit easier. I am not a maintainer on this repo sadly so can't take away the pain.