Skip to content

Newcastle#961

Open
chriswyatt1 wants to merge 19 commits intonf-core:masterfrom
chriswyatt1:newcastle
Open

Newcastle#961
chriswyatt1 wants to merge 19 commits intonf-core:masterfrom
chriswyatt1:newcastle

Conversation

@chriswyatt1
Copy link
Contributor


name: New Config
about: A new cluster config

Please follow these steps before submitting your PR:

  • If your PR is a work in progress, include [WIP] in its title
  • Your PR targets the master branch
  • You've included links to relevant issues, if any

Steps for adding a new config profile:

  • Add your custom config file to the conf/ directory
  • Add your documentation file to the docs/ directory
  • Add your custom profile to the nfcore_custom.config file in the top-level directory
  • Add your profile name to the profile: scope in .github/workflows/main.yml
  • OPTIONAL: Add your custom profile path and GitHub user name to .github/CODEOWNERS (**/<custom-profile>** @<github-username>)

@chriswyatt1
Copy link
Contributor Author

Thanks James. We will speak to the HPC admins this week at Newcastle University (UK). To ensure all those issues are resolved.

Added comments to clarify config parameters.
Updated Newcastle HPC configuration documentation with detailed usage notes, installation steps for Java and Nextflow, and troubleshooting tips.
Updated documentation to reflect the name change from Singularity to Apptainer and specified its availability on the comet system.
Added in correct resource limits section
Copy link
Member

@jfy133 jfy133 left a comment

Choose a reason for hiding this comment

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

Preemptive ✔️ once the relevant URL is added to the config description.

In the meantime @chriswyatt1 you could probably just point to the Newcastle University's homepage

Add URL to Newcastle. 
Would be better to the actual docs of comet, but this is here for now.
Add specific that its UK Newcastle University
Pseudonym: University of Newcastle upon Tyne
Add link to official page in docs
@chriswyatt1
Copy link
Contributor Author

HPC administration group will do checks on Monday. So hopefully then we can merge it. Thanks James!

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