Skip to content

Rename TRE GPU Service to SHS GPU Cluster and Add to Main Documentation - #343

Open
umairayub38 wants to merge 3 commits into
EPCCed:mainfrom
umairayub38:main
Open

Rename TRE GPU Service to SHS GPU Cluster and Add to Main Documentation#343
umairayub38 wants to merge 3 commits into
EPCCed:mainfrom
umairayub38:main

Conversation

@umairayub38

Copy link
Copy Markdown
Contributor

EIDF Documentation Pull Request

Description

Updated the GPU cluster documentation to reflect the new SHS GPU Cluster naming and integrated the documentation into the main EIDF documentation navigation.

The changes include:

  • Renamed the existing tre-gpu-service documentation directory to shs-gpu-cluster.
  • Updated references and page titles from TRE GPU Service / TRE GPU Cluster to SHS GPU Cluster where appropriate.
  • Added the SHS GPU Cluster documentation to the main mkdocs.yml navigation.
  • Fixed a minor trailing whitespace issue in the Cirrus documentation.

Dependencies:

  • No new dependencies are required.

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • [ x] Incorrect Documentation
  • Incomplete Documentation
  • Missing Documentation
  • [x ] Formatting
  • New Documentation

What has to be reviewed

  • Main EIDF documentation navigation in mkdocs.yml.
  • Links and navigation between the SHS GPU Cluster pages.
  • Terminology changes from TRE GPU Service to SHS GPU Cluster.

Checklist

  • Documentation follows the project style guidelines
  • Ensure Contact details contain Service Emails and Numbers
  • Self-review of documentation using mkdocs on local system
  • Spellcheck has been performed
  • Pre-commit has been run and passed

@kavousan
kavousan self-requested a review August 27, 2026 15:50

@kavousan kavousan 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.

I am not sure about the Cirrus page.

I made some suggestions about renaming more files that say TRE; please let me know if too onerous to find all their occurrences and fix them.

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.

Please rename to ... in-the-EPCC-SHS.md

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

page links are updated

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.

Please rename to ... in-the-EPCC-SHS.md

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

page links are updated

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.

Am I allowed to approve Cirrus changes? I am not sure...

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

When i run the pre-commit on repo in auto fix this file, i think there is a space in doc

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

change is reverted

@umairayub38
umairayub38 requested a review from kavousan August 28, 2026 13:22
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