FunGeST / Palimpsest

An R package for studying mutational signatures and structural variant signatures along clonal evolution in cancer.
69 stars 19 forks source link

compare_results() not working. "`template` must have a `names` attribute." #63

Closed subashani01 closed 5 months ago

subashani01 commented 8 months ago

compare_tab <- compare_results(reference_sigs = SBS_cosmic, extraction_1 = SBS_denovo_sigs); compare_tab Error: Internal error: template must have a names attribute.

While running the compare_results(), it shows me this error. I have checked the attributes of both inputs and both have names attribute. Please help me resolving this.

I have tried with your test data as well and getting the same error.

FunGeST commented 8 months ago

Dear @subashani01, I have just run the test script with the test data on my personal computer (Palimpsest 2.0.0, R 4.1.3), and I have no error when running compare_tab <- compare_results(reference_sigs = SBS_cosmic, extraction_1 = SBS_denovo_sigs). Have you tried to run it again with a fresh R environment to avoid interactions with other loaded packages?

subashani01 commented 8 months ago

Thank you so much for replying. Yes, I have. Also, I am using R 4.3.2

nyoonhee commented 8 months ago

I am having the exact same error message. I am using Palimpsest_2.0.0, R 4.2.1.

subashani01 commented 7 months ago

Can R version be the cause of error?

benedict909 commented 6 months ago

Dear all,

Firstly thank you for using Palimpsest!

I have just encountered this error for the first time, as @subashani01 suggests this seems to be an error that occurs with newer versions of R, although this was in no part helped by my dodgy old code!

I have fixed this error, although unfortunately I dont currently have access to the @FunGeST account so can't merge myself (if someone sees this please feel free to merge my pull request #57 !), but you can download the fixed code from my github:

devtools::install_github("benedict909/Palimpsest")

Let me know if you have any more issues 😄

Benedict

FunGeST commented 5 months ago

Thank you @benedict909, your pull request #57 has been merged!

FunGeST commented 5 months ago

The fix provided by @benedict909 should resolve the issue, it has been merged to the main.