pneuvial / c3co

Inferring cancer cell clonality from copy-number data
5 stars 1 forks source link

BUG: cat(str(x)) #15

Closed HenrikBengtsson closed 7 years ago

HenrikBengtsson commented 7 years ago

FYI, using cat(str(...)) is most likely a bug, e.g. from https://github.com/pneuvial/c3co/blob/master/R/AllGenerics.R#L22-L26:

        cat("Subclones\n")
        cat(str(this@S), "\n")
        cat("Weights\n")
        cat(str(this@W), "\n")
        cat("Estimates\n")
        cat(str(this@E), "\n")
        [...]
        cat("param\n")
        cat(str(this@param), "\n")
pneuvial commented 7 years ago

Fixed in efa5200e8d7db4b4a0b31943aeb865624e8d29b0

Also renamed the showPosFused and showC3co methods to 'show'.