search

kevinblighe / EnhancedVolcano

Publication-ready volcano plots with enhanced colouring and labeling
399 stars 81 forks source link

Option for rasterization #53

Closed benostendorf closed 4 years ago

benostendorf commented 4 years ago

Thanks a lot for a great package!

Given the large number of datapoints on a typical volcano plot, these plots often become sluggish to deal with. Therefore, it would be fantastic if there was an option to rasterize the output.

One way I could imagine it would be to have an option to set raster = TRUE in the function. This would call geom_point_rast from the ggrastr package rather than geom_point.

Edit: I submitted a pull request for this. Thanks for considering it!

kevinblighe commented 4 years ago

Hi @benostendorf , thanks for the contribution! It will take me some time before I can look through these Pull Requests in great detail.

kevinblighe commented 4 years ago

Hey @benostendorf, thanks for these - I have now incorporated them into the main branch and they will propagate to the Bioconductor Devel branch in the coming days. You are also added as Contributor.

If you could do further testing with raster (before the next official Bioc release), that would be great!

Closed in https://github.com/kevinblighe/EnhancedVolcano/pull/55