-
Notifications
You must be signed in to change notification settings - Fork 708
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Backport support for Graviton 4 to version 3.16.1 #1414
Conversation
Warning Newer version of the nf-core template is available. Your pipeline is using an old version of the nf-core template: 2.14.1. For more documentation on how to update your pipeline, please see the nf-core documentation and Synchronisation documentation. |
63eb1ec
to
4eebdf5
Compare
Signed-off-by: Pablo <[email protected]>
nextflow.config
Outdated
@@ -123,7 +123,7 @@ params { | |||
|
|||
// Max resource options | |||
// Defaults only, expecting to be overwritten | |||
max_memory = '128.GB' | |||
max_memory = '256.GB' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we revert this change too @pabloaledo ? Shouldn't make a difference setting this higher.
Sw to conf
@drpatelh is there already an upstream PR for this? I mean merging the branch |
@pditommaso not yet- I'm basically working to cut down what's necessary here. Almost done. |
PR checklist
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).nextflow run . -profile debug,test,docker --outdir <OUTDIR>
).docs/usage.md
is updated.docs/output.md
is updated.CHANGELOG.md
is updated.README.md
is updated (including new tool citations and authors/contributors).