miRTop / mirtop

command lines tool to annotate miRNAs with a standard mirna/isomir naming
https://mirtop.readthedocs.org
MIT License
18 stars 21 forks source link

Plot of isomiRs/miRNA distribution using ggplot-python #21

Open lpantano opened 6 years ago

lpantano commented 6 years ago

Implement option in stats sub-command like --detail to:

[ ] show distribution of sequences per sample (geom_density) [ ] boxplot distribution per sample after normalization like DESeq2 [ ] summary figure like isoPlot(type = "all") from isomiRs bioc package

cranewen commented 6 years ago

Hi Lorena,

I have been working on the plot. I couldn't make the plot correctly in ggplot. It seems missing y axis values. But it works for dots or lines graphs. It looks like that. Please let me know if you have any idea on that.

screen shot 2018-01-20 at 12 38 52 pm

Or I can make it with Plotly, it looks like this: newplot

What do you think?

lpantano commented 6 years ago

Hi,

That seems a good format as well. But I see all of them looks like 1, is that a typo? If we the right numbers as it is in the example code, you can make a PR to the code adding the code inside the mirtop/plot folder.

Thanks a lot!

On Jan 20, 2018, at 12:42 PM, cranewen notifications@github.com wrote:

Hi Lorena,

I have been working on the plot. I couldn't make the plot correctly in ggplot. It seems missing y axis values. But it works for dots or lines graphs. It looks like that. Please let me know if you have any idea on that. https://user-images.githubusercontent.com/9448300/35186254-33e64cb4-fddf-11e7-81da-c52bf911b15b.png Or I can make it with Plotly, it looks like this: https://user-images.githubusercontent.com/9448300/35186219-aa0c5a60-fdde-11e7-8ef3-f1c399a58e72.png What do you think?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/miRTop/mirtop/issues/21#issuecomment-359188681, or mute the thread https://github.com/notifications/unsubscribe-auth/ABi_HDIY5te9llFrFPg1qsuCIWxQgtc8ks5tMiWZgaJpZM4RjuwP.

cranewen commented 6 years ago

Hi Lorena,

No, it's not a typo. It's the problem I am facing now. If I use line or point plot, it shows below, which is what I expected. The code is ggplot(df, aes(x='tool',y='counts')) + geom_line() + facet_wrap('category')

screen shot 2018-01-23 at 8 32 34 am

But in the bar chart, ggplot(df, aes(x='tool',y='counts')) + geom_bar() + facet_wrap('category') I don't think y='counts' works in the function.

I will pull request once I fixed this problem. If you know what's wrong here, please let me know.

Thanks, Crane

lpantano commented 6 years ago

Thanks for working on this.

Can you try to add this option inside geom_bar function: stat = “identity",

And see if that fixes the issue.

Cheers

On Jan 23, 2018, at 8:57 AM, Crane Wen notifications@github.com wrote:

Hi Lorena,

No, it's not a typo. It's the problem I am facing now. If I use line or point plot, it shows below, which is what I expected. The code is ggplot(df, aes(x='tool',y='counts')) + geom_line() + facet_wrap('category') https://user-images.githubusercontent.com/9448300/35278883-622fac6e-0019-11e8-82d9-66534758088e.png But in the bar chart, ggplot(df, aes(x='tool',y='counts')) + geom_bar() + facet_wrap('category') I don't think y='counts' works in the function.

I will pull request once I fixed this problem. If you know what's wrong here, please let me know.

Thanks, Crane

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/miRTop/mirtop/issues/21#issuecomment-359798028, or mute the thread https://github.com/notifications/unsubscribe-auth/ABi_HPOR1h7oanVxADXW2K0EIXxX_sxvks5tNeU4gaJpZM4RjuwP.

cranewen commented 6 years ago

No, it's the same.

lpantano commented 6 years ago

Other option is to plot a point instead with geom_point and increase the size a little. We can try to figure out the issue later as well.

Thanks!

On Jan 23, 2018, at 9:24 AM, Crane Wen notifications@github.com wrote:

No, it's the same.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/miRTop/mirtop/issues/21#issuecomment-359806035, or mute the thread https://github.com/notifications/unsubscribe-auth/ABi_HKiZIe_OrS-_9Plw1pzKWoHM-iIpks5tNeu4gaJpZM4RjuwP.

cranewen commented 6 years ago

Sure, Lorena. But I will try to find out a way to plot it in bar charts tonight. Do you want me to pull request? I also did column 9 of GFF and return key #25. I am gonna keep working on all the issues. I now have time during the nights, and I am learning the whole project slowly:)

lpantano commented 6 years ago

Yes, please, one pull request per issue. And please, update the REAMDE.md with your name in the contributors section.

Thanks