lima1 / PureCN

Copy number calling and variant classification using targeted short read sequencing
https://bioconductor.org/packages/devel/bioc/html/PureCN.html
Artistic License 2.0
127 stars 32 forks source link

plotting error with cell line samples and --min-purity 0.98 --max-purity 0.99 #344

Closed ThomasGro closed 8 months ago

ThomasGro commented 8 months ago

with the following settings I get a plotting error for all my ~400 cell line samples and Mutect2 VCF:

--genome GRCh38
--min-af 0.03
--min-purity 0.98
--max-purity 0.99
--min-ploidy 1.4
--max-ploidy 6.0
--max-copy-number 7
--max-segments 300
--max-non-clonal 0.2
--min-total-counts 100
--model-homozygous 
--model beta
--fun-segmentation CBS
--post-optimize 
--seed 42 

example SDTOUT,SDTERR: ... Generating output files... Warning message: In .bcfHeaderAsSimpleList(header) : duplicate keys in header will be forced to unique rownames Error in plot.window(...) : need finite 'ylim' values Calls: plotAbs ... .plotTypeBAF -> plot -> plot.default -> localWindow -> plot.window In addition: Warning messages: 1: In .Call(.make_vcf_geno, filename, fixed, names(geno), as.list(geno), : converting NULL pointer to R NULL 2: In plot.default(xLogRatio, logRatio$log.ratio, ylab = "Copy Number log-ratio", : NaNs produced Execution halted

PureCN did not finish successfully for sample CELL_143B!

Any idea what the problem could be or how I can track the problem?

lima1 commented 8 months ago

Hmm, can't remember seeing this one before. Not sure it's related, but do you run into (other) issues with --genome hg38? PureCN pulls some genome specific annotation files from Bioconductor, and not sure GRC38 works here. I'm might have addressed it, but can't remember.

If not, can you share the full log file of this sample?

lima1 commented 8 months ago

Actually, before you change anything, can you make sure that you use the latest stable version? I vaguely remember fixing a crash similar to that.

ThomasGro commented 8 months ago

Unfortunately this version is part of a workflow systen and cannot easily and quickly be updated. But I have received a hint in the same direction from a colleague: a newer version of PureCN fixed the issue. Thank you for following up. From: Markus Riester @.> Sent: Samstag, 20. Januar 2024 00:51 To: lima1/PureCN @.> Cc: Thomas Grombacher @.>; Author @.> Subject: Re: [lima1/PureCN] plotting error with cell line samples and --min-purity 0.98 --max-purity 0.99 (Issue #344)

[WARNING - EXTERNAL EMAIL] Do not open links or attachments unless you recognize the sender of this email. If you are unsure please click the button "Report suspicious email"

Actually, before you change anything, can you make sure that you use the latest stable version? I vaguely remember fixing a crash similar to that.

— Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/lima1/PureCN/issues/344*issuecomment-1901329507__;Iw!!Eu8ikxSnpXkBCg!aJo6PkydWSNamcH8mMI54rKJo85cfBTcad97aivcoXo6BOB1RwbgAUdkViKSwNklfEZmbCR1xZeOGp1dyP7IWU0-o04MsjMZRUA$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/AENMOYNGQXXXZ6EGYC6ZPZDYPMBE3AVCNFSM6AAAAABCB5V4UOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBRGMZDSNJQG4__;!!Eu8ikxSnpXkBCg!aJo6PkydWSNamcH8mMI54rKJo85cfBTcad97aivcoXo6BOB1RwbgAUdkViKSwNklfEZmbCR1xZeOGp1dyP7IWU0-o04MG0igFKw$. You are receiving this because you authored the thread.Message ID: @.**@.>>

This message and any attachment are confidential and may be privileged or otherwise protected from disclosure. If you are not the intended recipient, you must not copy this message or attachment or disclose the contents to any other person. If you have received this transmission in error, please notify the sender immediately and delete the message and any attachment from your system. Merck KGaA, Darmstadt, Germany and any of its subsidiaries do not accept liability for any omissions or errors in this message which may arise as a result of E-Mail-transmission or for damages resulting from any unauthorized changes of the content of this message and any attachment thereto. Merck KGaA, Darmstadt, Germany and any of its subsidiaries do not guarantee that this message is free of viruses and does not accept liability for any damages caused by any virus transmitted therewith.

Click merckgroup.com/disclaimerhttps://www.merckgroup.com/en/legal-disclaimer/mail-disclaimer.html to access the German, French, Spanish, Portuguese, Turkish, Polish and Slovak versions of this disclaimer.

Please find our Privacy Statement information by clicking here: merckgroup.com/privacy-statements-by-locationhttps://www.merckgroup.com/en/privacy-statement/privacy-statements-by-location.html

lima1 commented 8 months ago

Looks like this is the previous bug: https://github.com/lima1/PureCN/issues/260

Maybe try reducing the max-ploidy to something lower if you can't update now. 6 should be super rare.