-
Notifications
You must be signed in to change notification settings - Fork 642
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
New config option: publishDir.basePath
#3805
Comments
Related to #2661 |
This comment was marked as outdated.
This comment was marked as outdated.
This has been solved by #3942. In a nutshell, relative paths are resolved against the path specified via |
There's no way to set this from within the workflow though, right? We need to be able to set this with |
This may be a good candidate for |
Yeah, it's to prefix rather than overwrite 🤔 |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This should be addressed by the workflow outputs DSL #4784 which removes the need to specify the base publish path many times |
New feature
In @nf-core we prefix every
publishDir
path
withparams.outdir
. For example, to modify the example in the docs:publishDir "${params.outdir}/data/chunks"
Without this, Nextflow will spit published results into the launch directory, which I think is generally agreed to be a bad idea in most cases (overwriting / ephemeral storage). But to follow best practices of not doing this, the pipeline code needs this arbitrary variable interpolation.
It would be nice to have a new config attribute for
publishDir
called something likebasePath
, which would default to the launch directory (existing behaviour) but if set, works for the entire pipeline run.This would remove quite a lot of boilerplate code and make it easier for people working with non-nf-core pipelines which don't use this convention.
Could also have this as a command-line flag to easily change at run time, similar to
-work-dir
(-publish-dir
?)Suggest implementation
publishDir.basePath
-publish-dir
If provided, prefix this string to the path in
PublishDir.groovy
The text was updated successfully, but these errors were encountered: