Skip to content

Add singularity configuration to bactmap#994

Closed
iamamofa wants to merge 1 commit intomasterfrom
iamamofa-patch-1
Closed

Add singularity configuration to bactmap#994
iamamofa wants to merge 1 commit intomasterfrom
iamamofa-patch-1

Conversation

@iamamofa
Copy link
Copy Markdown


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

Comment thread pipeline/bactmap.config
Comment on lines +14 to +20
// using singularity platform
singularity {
enabled = true
autoMounts = true
cacheDir = ''
pullTimeout = '*h'
}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

While I typically think you should use singularity/apptainer, having this in the pipeline configuration seems odd and likely to break stuff?

I don't think I understand the intention here.

@jfy133
Copy link
Copy Markdown
Member

jfy133 commented Jan 22, 2026

These changes are already set in the core nf-core template (minus the pull timeout) but that should be updated on a per-cluster/infra basis, not globally.

I will close this, but please shout onthe nf-core Slack @iamamofa if you need more help

@jfy133 jfy133 closed this Jan 22, 2026
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