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

Changing max_rho from 1 to 0.9 will break ascat.plotSunrise() #38

Closed
TakaYamaguchi opened this issue Jun 5, 2019 · 5 comments
Closed
Assignees
Labels
for v3.0 Change to be included in version 3.0

Comments

@TakaYamaguchi
Copy link

Hi,

As sometimes purity seems to be overestimated, I tried max_rho = 0.9 but it caused the issue below.

Error in axis(2, at = seq(0, 1/purity_max, by = 1/3/purity_max), labels = seq(purity_min, :
'at' and 'labels' lengths differ, 4 != 3
Calls: fit.copy.number -> runASCAT -> -> axis

I'm currently testing min_rho = 0, which may fix the issue but I think the axis labels should handle different min max rho values...?

Thanks,

Taka

@TakaYamaguchi TakaYamaguchi changed the title Changing max_rho 0.9 from 1 will break ascat.plotSunrise() Changing max_rho from 1 to 0.9 will break ascat.plotSunrise() Jun 5, 2019
@TakaYamaguchi
Copy link
Author

Just letting you know that setting min-rho = 0, max-rho = 0.9 didn't help..

@TakaYamaguchi
Copy link
Author

Hi,
The program worked fine when the Sunrise plot was commented out in clonal_ascat.R
Line1432 - ASCAT::ascat.plotSunrise

@lydiayliu
Copy link

T_T im running into the same issue. Also setting min_ploidy and max_ploidy to force WGD breaks the sunrise plotting.

axis(1, at = seq(0, 1, by = 1/(ploidy_max-1)), labels = seq(ploidy_min, ploidy_max, by = 1))

For example ploidy_max = 4.8 and ploidy_min = 2.9 will result in "by" having length of 4 and "labels" having length of 2...

@lydiayliu
Copy link

For my own record, to update the Battenberg:::runASCAT() function in the docker without having to rebuild the package and the docker, I did

Create a /mnt/scripts/runASCAT_hack.R, with
runASCAT.battenberg.nosunrise <- function(...) { ... #ASCAT::ascat.plotSunrise ...}

When running Battenberg

library(Battenberg)
source('/mnt/scripts/runASCAT_hack.R')
environment(runASCAT.battenberg.nosunrise) <- asNamespace('Battenberg')
assignInNamespace("runASCAT", runASCAT.battenberg.nosunrise, ns = 'Battenberg', envir=as.environment('package:Battenberg'))

@jcesar101 jcesar101 self-assigned this Feb 21, 2023
@jcesar101 jcesar101 added the for v3.0 Change to be included in version 3.0 label May 31, 2023
@jcesar101
Copy link
Contributor

Apologies for not replying before. Although this problem is not directly related to Battenberg, and that it currently doesn't allow the user to modify the max_rho value (only programatically), we have tested this scenario and it's working fine with max_rho different from 1.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
for v3.0 Change to be included in version 3.0
Projects
None yet
Development

No branches or pull requests

3 participants