Closed emirerkol closed 3 years ago
Hey @emirerkol , thanks for reporting this. Indeed, you already figured out that you cannot put a full path into this parameter. My first thought would be to say "you're using it wrong" by doing this, but I guess that it is not really clear, so probably not fair to blame you entirely. There should have been a default value in there already when you generated the config and I usually don't change it. I wonder if maybe we should remove these parameters altogether and let the pipeline determine the output file names (which is what happens in the other workflows).
Hi @cflerin, indeed maybe the best option is to remove the parameters (or maybe a warning to not give a full path).
Describe the bug A clear and concise description of what the bug is.
When given a path for the loom files to be published (defined in .config file), an error occurs at the SCENIC:MERGE_MOTIF_TRACK_LOOMS step if an absolute path outside the Nextflow working directory is defined. This can be easily avoided if only a filename is given so the file can be published inside the Nextflow working directory.
To Reproduce Steps to reproduce the behavior:
Make a Nextflow .config file for scenic.
Define paths for
scenicOutputLoom
,scenicScopeOutputLoom
(only important part in the config file is given).Expected behavior The error comes up pretty late in the process thus the defined path may be checked after when the Nextflow working directory is first created for the potential error. Maybe a default file name can be assigned instead of the given output in the .config file.
Please complete the following information:
Additional context This issue can be avoided if just a filename is given for the output files like given in the config file as default.