-
Notifications
You must be signed in to change notification settings - Fork 41
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
Adding MinIONQC as module to MultiQC #40
Comments
Sorry for being so slow - teaching! I'll have a think about how best to do this. I know that in principle I can just grab the data out of the relevant ggplot objects. The issue is that these files will often be ridiculously large. That got me thinking that I should downsample them, and if I'm going to do that I should actually rewrite a lot of the code fairly substantially to speed everything up. But I'm working on it! |
And also, I should say - thanks for doing this! The current iteration looks great, and I absolutely agree that multiqc is awesome, so I'm very keen to push forward with this integration. |
I'm glad that you find this useful, and thanks for being open to the idea. I made some improvements to the module and submitted a pull request to multiqc. Here is how it looks now: multiqc_report.html.zip. Any suggestions on improvements or help-text edits on the report are welcome! I briefly spent time last week on how to downsample/subsample the data while retaining the needed info for the plot, but I didn't get too far; partly because R is not my strong suit. If I find anything useful on this front, I will send them your way. |
Has MultiQC now been updated that it includes displaying the plots output by MinIONQC as well as the stats from summary.yaml? |
@ManavalanG - the MultiQC docs don't make it sound like the plots are included. If it's still of interest, I can make sure to output the plot data. |
Hi @roblanf! In my original pull request, I had plots for reads N50, median q and base count, but multiqc's author had replaced them with just readlength plot. This is probably why. This PR was handled and merged two months after I submitted it and by then I didn't have time to look into the changes made. I found this paper in the wild in which figure 3 shows how the tables look in multiqc (doesn't show the plot though). It's been a while since I worked with nanoplot data, but if you are interested in improving minionqc integration into multiqc, I am open to collaboration. |
Just realized that you were talking about major plots (length_by_hour, etc.) produced by minionqc and not something simple like base count for which tabulation might just be sufficient. So yes, if minionqc outputs simplified data as originally requested in this issue, we could add them to multiqc in few weeks I would think. |
Hello @ManavalanG @roblanf, |
Reopening this issue, with the hopes that I can get to it soon. Fair warning though - you can see already that it's taken me a long time, and right now I (like a lot of people) have my focus predominantly elsewhere. |
@roblanf is there a way i can get raw data(.tsv or .csv) used to plot all the graphs by minionqc? |
Hi, I am working on including support for MinIONQC in MultiQC, which is a fantastic results aggregator. So far, I have added results using
summary.yaml
; see this html file for its current state - multiqc_report.html.zip. Your feedback is welcome.But what would be truly useful is including plots below, as this would enable easier comparison of plots across samples.
This would require raw data output for above plots from MinIONQC. It would be great if you could enable writing relevant plot raw data as output (as csv, json, etc).
PS:
If you would like to try my dev version of MultiQC, follow these steps:
cd MultiQC
python setup.py develop
multiqc <dir_with_minionqc_results>
multiqc_report.html
The text was updated successfully, but these errors were encountered: