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

blank_genome: an alternative code #13

Closed
qingjian1991 opened this issue Nov 1, 2021 · 4 comments
Closed

blank_genome: an alternative code #13

qingjian1991 opened this issue Nov 1, 2021 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@qingjian1991
Copy link
Contributor

Hi:

I made some changes on function blank_genome() for better visualizations on CNV plot, please review my code.

The figure before changes, The 1q has a complex CNAs changes:

image

The figure after changes:

image

The changed code is here:
blank_genome.R

Thanks.

Qingjian Chen

@caravagn
Copy link
Collaborator

caravagn commented Nov 3, 2021

Hi @qingjian1991 ,

Thanks for the message.

Is the only change that you are doing this?

    #set the chr names in the centromer positions.
    scale_x_continuous(
      breaks = c(0, reference_coordinates$centromerStart, upp),
      labels = c("", gsub(pattern = 'chr', replacement = '', reference_coordinates$chr), "")
    )

Could you show with some plots how does this visualise when you do not follow the circular layout (i.e., as a single sample)?

Btw, we'd have to go for a PR if we accept this modification.

@caravagn caravagn self-assigned this Nov 3, 2021
@caravagn caravagn added the enhancement New feature or request label Nov 3, 2021
@qingjian1991
Copy link
Contributor Author

Is the only change that you are doing this?

I made two main changes to this code. One is as your notice.

Another change is here. The origin code used dashed lines to separate the chromosomes and solid lines to indicate the positions of the centromere. The modified code changed the orders, by using the solid lines to separate the chromosomes and dashed lines to indicate the position of the centromere.

image

@caravagn
Copy link
Collaborator

I think it is OK, open a PR if you want me to get this into the code.

@caravagn
Copy link
Collaborator

caravagn commented Sep 1, 2022

Merged #16 thanks @caravagn

@caravagn caravagn closed this as completed Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants