Skip to content

chr.sub #4

Open
Open

Description

@walaj

Propose removing this. Not sure what the issue is with having "chr" in the seqinfo, and it seems like if there is a problem, a more general solution should be used (e.g. manipulate seqinfo however you like on your own, and then set with seqinfo<- )

Activity

  1. mskilab commented on Mar 20, 2016

    @mskilab
    Collaborator

    sure agree in principle

    this basically applies to gTrack that we build off external files - e.g. .bam, .wig., .bw etc where you can't manipulate the seqinfo in that manner

    in practice "chr" are a <> since a lot of UCSC tracks will be stored as .bw with chr prefixes and then you're trying to plot them against your broad .bam which has no "chr" ...

    open to other fixes .. but we should have a solution for this

  2. KhagayN commented on Dec 5, 2016

    @KhagayN
    Collaborator

    if chr.sub is removed, how would we remove the "chr" prefixes from external files such as .bam, .wig, .bw where they can't be manipulated by using the seqinfo function?

  3. mskilab commented on Dec 6, 2016

    @mskilab
    Collaborator
  4. mskilab commented on Dec 6, 2016

    @mskilab
    Collaborator
  5. KhagayN commented on Dec 6, 2016

    @KhagayN
    Collaborator

    ahh, especially as you mentioned on Mar 20, when plotting these tracks against broad .bam that has no "chr". Sounds like keeping chr.sub is a good idea but, include tests that chr.sub works for the external files that can be inputted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions