Skip to content

Update NCI Gadi config#920

Open
georgiesamaha wants to merge 9 commits intonf-core:masterfrom
Sydney-Informatics-Hub:nci_gadi
Open

Update NCI Gadi config#920
georgiesamaha wants to merge 9 commits intonf-core:masterfrom
Sydney-Informatics-Hub:nci_gadi

Conversation

@georgiesamaha
Copy link
Contributor

@georgiesamaha georgiesamaha commented Jun 25, 2025


name: NCI Gadi HPC
about: added new instructions re: use of persistent sessions to docs, directed singularity cacheDir to /scratch to avoid issues with defaulting to /home

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>)

@georgiesamaha
Copy link
Contributor Author

Included fixes from #872

singularity {
enabled = true
autoMounts = true
cacheDir = "/scratch/${params.project}/singularity"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Does this exist already, writable for everyone with the same project set?

executor = 'pbspro'
project = 'aa00'
storage = 'scratch/aa00+gdata/aa00'
storage = 'scratch/params.project'
Copy link
Collaborator

Choose a reason for hiding this comment

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

I see there's apparently since before the "recommendation" to make a copy of the config file rather than just overriding what's needed (as typically done, e.g. as referenced from https://nf-co.re/docs/usage/getting_started/configuration).

If one does, this should have an expansion I assume? (Or could be hardcoded if a user does her own config).

Copy link
Collaborator

Choose a reason for hiding this comment

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

So ${params.project} rather than params.project?

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