Skip to content
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

Dropulation DetectDoublets execution error #56

Open
lhoranportelance opened this issue Sep 11, 2024 · 3 comments
Open

Dropulation DetectDoublets execution error #56

lhoranportelance opened this issue Sep 11, 2024 · 3 comments

Comments

@lhoranportelance
Copy link

Hi again @drneavin :)

I got the error I was having on AssignCellsToSamples fixed yesterday. However, I'm now having a different error when running the DetectDoublets step of Dropulation. It seems to be the same thing you were experiencing here with AssignCellsToSamples: broadinstitute/Drop-seq#422

I'm getting the same "Illegal argument value: Positional arguments were provided" error for the first variable in my command. I was running my command with the options set as --OPTION . I tried switching to OPTION=, but it then gave the "positional argument" error on every option in the command rather than just the first one. It's especially weird because I didn't run into this issue with AssignCellsToSamples--it's only popped up with DetectDoublets. I'm not sure if you maybe changed something in AssignCellsToSamples but not DetectDoublets? Or is there a fix that I'm not entirely understanding from that other thread?

Thanks in advance :)
Liam

@drneavin
Copy link
Owner

Thi @lhoranportelance , thanks for your post. Can you provide the output from singularity inspect Demuxafy.sif? I want to see what version of the singularity image you have installed. Thanks!

-Drew

@lhoranportelance
Copy link
Author

@drneavin here's the output:

singularity inspect Demuxafy.sif
Author: = Drew Neavin
Image_version: = v3.0.0
Software: = Demuxafy
org.label-schema.build-arch: amd64
org.label-schema.build-date: Tuesday_4_June_2024_14:1:56_UTC
org.label-schema.schema-version: 1.0
org.label-schema.usage.singularity.deffile.bootstrap: library
org.label-schema.usage.singularity.deffile.from: ubuntu:20.04
org.label-schema.usage.singularity.version: 3.11.3-dirty

and so you have it, here was the command I executed that did work:

singularity exec --bind /data/... Demuxafy.sif Dropulation_AssignCellsToSamples.py --CELL_BC_FILE $BARCODES \
    --INPUT_BAM $DROPULATION_OUTDIR/possorted_genome_bam_dropulation_tag.bam \
    --OUTPUT $DROPULATION_OUTDIR/assignments.tsv.gz \
    --VCF $VCF_ALL \
    --SAMPLE_FILE $INDS \
    --CELL_BARCODE_TAG $CELL_TAG \
    --MOLECULAR_BARCODE_TAG $UMI_TAG \
    --VCF_OUTPUT $DROPULATION_OUTDIR/assignment.vcf \
    --CREATE_INDEX true \
    --MAX_ERROR_RATE 0.05

and here was the command that didn't work:

singularity exec --bind /data/... Demuxafy.sif DetectDoublets --CELL_BC_FILE $BARCODES \
    --INPUT_BAM $DROPULATION_OUTDIR/possorted_genome_bam_dropulation_tag.bam \
    --OUTPUT $DROPULATION_OUTDIR/likelihoods.tsv.gz \
    --VCF $DROPULATION_OUTDIR/assignment.vcf \
    ${CELL_TAG:+--CELL_BARCODE_TAG $CELL_TAG} \
    ${UMI_TAG:+--MOLECULAR_BARCODE_TAG $UMI_TAG} \
    --SINGLE_DONOR_LIKELIHOOD_FILE $DROPULATION_OUTDIR/assignments.tsv.gz \
    --SAMPLE_FILE $INDS \
    --MAX_ERROR_RATE 0.05

(changed file paths for HPC)

@lhoranportelance
Copy link
Author

@drneavin I believe I may have solved my own problem again, but I'm not sure why or how it worked.

All I did was add "--" after "DetectDoublets" in the command (as suggested here: broadinstitute/Drop-seq#422 (comment)), and it seems to be working.

singularity exec --bind /data/... Demuxafy.sif DetectDoublets -- --CELL_BC_FILE $BARCODES \
    --INPUT_BAM $DROPULATION_OUTDIR/possorted_genome_bam_dropulation_tag.bam \
    --OUTPUT $DROPULATION_OUTDIR/likelihoods.tsv.gz \
    --VCF $DROPULATION_OUTDIR/assignment.vcf \
    ${CELL_TAG:+--CELL_BARCODE_TAG $CELL_TAG} \
    ${UMI_TAG:+--MOLECULAR_BARCODE_TAG $UMI_TAG} \
    --SINGLE_DONOR_LIKELIHOOD_FILE $DROPULATION_OUTDIR/assignments.tsv.gz \
    --SAMPLE_FILE $INDS \
    --MAX_ERROR_RATE 0.05

I really have no idea why it was required for this command but I didn't have to do it for AssignCellsToSamples. May be something to look into for documentation purposes.

Apologies for the confusion!
Liam

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

No branches or pull requests

2 participants