-
Notifications
You must be signed in to change notification settings - Fork 420
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
The sample-sheet only contains tumor-samples #1260
Comments
Can you share the |
Here you go: |
I guess it would also be relevant for us to inspect the sample sheet. Can you also share that with us? |
Yeah, I think the issue is unrelated to that error. |
Same error here using nextflow-23.04.4-all
|
Can you share the log file please? |
I am also getting the same error even though my sample sheet contains only normal samples ( |
can you share the log file? |
I also got the |
@bounlu Can you try without |
Yeah, I think the 2 are fully unrelated, but no idea why the |
Without |
this is my "launcher":
All my samples are germline and I am not annotating. This is what I get:
|
ok, so https://github.com/nf-core/sarek/releases/tag/3.3.2 should fix these issues |
I still get similar error with the new version:
|
ok, so we get all error messages at once, but I think it all lies with this error:
|
in your case @bounlu I'd say that adding |
hey! also got the error
when starting
in my understanding the error is linked to the check in as does if i explicitly define the parameters (before tools) when starting the pipeline both comparisons will actually evaluate to true and no confusing error is raised (should be the same value as default, but maybe different type, a groovy expert might know):
|
Hello,
and it worked fine for me when I set:
or
instead of:
|
@achakroun can you send everything you used? Samplesheet, full command, the .nextflow.log, any custom configuration files? |
Sure 1- Command:
2- Samplesheet was like:
Nothing else. |
can you send the |
Please, note that the pipe is still running |
Sorry can you provide the log from the failed run? |
ah pretty sure I know what is wrong. opened a separate issue to fix --> #1314 |
this fixed the error for me. |
Hey! Yeah those are two completely unrelated issues, just nf-validation both times bubbles up |
@achakroun your issue is fixed on dev |
Hey, this problem has not been entirely fixed...
COMMAND:
Note, the pfr profile only specifies SLURM as the executor. pfr_params.jason:
Here is the error in the log file:
Here is the sample sheet: ` GC,1,GC_CAT13ANXX_TTAGGC,L001,/WGS/AGRF_CAGRF14129_CAT13ANXX/DH_A_GC_CAT13ANXX_TTAGGC_L001_R1.fastq.gz,/WGS/AGRF_CAGRF14129_CAT13ANXX/DH_A_GC_CAT13ANXX_TTAGGC_L001_R2.fastq.gz |
@charlesdavid can you send the file for the samplesheet? |
@charlesdavid @maxulysse It might be good to close this issue as the most recent comments are helping troubleshoot user issues. Leaving it in the open state makes it seem that there is still something that needs to be fixed with sarek. Perhaps it would be good to improve the clarity of that error message though as it still seems to cause confusion. |
This issue has been fixed, @charlesdavid your issue is now #1567. |
This bug emerged again after the latest merging of the
This was a resume from a previous successful run with the same command line. It worked then and now it fails again. |
I am guessing false flag from the error message (it for some reason shows up for unrelated issues). What is in the .nextflow.log & samplesheet |
What should be the correct format of the full path to the caches? There is also discrepancy between the documentation help text and the actual cache files located on igenomes S3: https://nf-co.re/sarek/3.4.3/parameters/#vep_cache says: S3 has: I am using a local cache and it seems I need to update the path format to make it work with the |
I think I also encountered that problem. Fixed it locally with some symlinks. |
Can you please share your directory tree for the cache dirs? |
We already had I suspect that @maxulysse changed the folder structure for the vep-cache (and snpeff-cache), but perhaps he can comment on that? |
Hi @maxulysse & @FriederikeHanssen, Thank you for your support on nf-core. I am still getting the "The sample-sheet only contains normal-samples, but the following tools, which were requested with "--tools", expect at least one tumor-sample : mutect2" error even though my samplesheet has tumor-normal pairs. I have attached the sample file and nextflow.log file. i used the commands: |
There is a compatibility bug with that version of nextflow and sarek 3.4.4 can you either downgrade nextflow to 24.04 or upgrade sarek? |
Description of the bug
I am getting a new type of error when running the same sample samplesheet as it used to work before:
The sample sheet contains both normal and tumor samples though.
Command used and terminal output
Relevant files
No response
System information
N E X T F L O W ~ version 23.08.1-edge
local
Docker
Linux
nf-core/sarek master v3.3.1
The text was updated successfully, but these errors were encountered: