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

Suggestion: link samples together? #28

Closed
alneberg opened this issue Sep 18, 2015 · 9 comments
Closed

Suggestion: link samples together? #28

alneberg opened this issue Sep 18, 2015 · 9 comments

Comments

@alneberg
Copy link
Contributor

Hi, just a humble suggestion can you start a new project named MetaMultiQC (suggested by @guillermo-carrasco) (or possibly MultiMultiQC).

Just kidding. Do you think its a good idea for support for linking samples together?

I'll try to explain what I mean:
My usecase is that I run fastqc for each sample at each step I take in the processing of the reads. Below you can see the results for three samples in two steps = 18 fastqc reports.
screen shot 2015-09-18 at 16 22 08

Would be nice to see a graph over how the stats change for a sample between the different steps for example. In order to do that I need a way to link the reports that belongs to the same sample together.

Thanks,
Johannes

@ewels
Copy link
Member

ewels commented Sep 18, 2015

I've thought about this before - unfortunately, I don't think that there's any safe way to do this without making prior assumptions about sample naming conventions (and so making the tool less general use). So I'm not planning to do this any time soon I'm afraid.

However - some good news: the data within the General Statistics table is also saved to a tab delimited file within the report, so it should be pretty easy to do this in excel.

Also, you could use highlighting patterns to pull out groups of samples for easier comparison (or hiding samples).

@ewels ewels closed this as completed Sep 18, 2015
@ewels ewels added the wontfix label Sep 18, 2015
@alneberg
Copy link
Contributor Author

Yes I considered doing it automatic as well and then you'll run into that issue. But couldn't it be possible to do on a user defined basis? Maybe a regexp entered by the user?

@ewels
Copy link
Member

ewels commented Sep 18, 2015

Maybe... If you can think of how it could work then I'll think about how to build it? I still think that it would be quite a lot of work for a fairly niche use case.

See also #29 which I was thinking about earlier today - might be possible to write a plugin at generation time which would be able to use this data with that approach..?

@alneberg
Copy link
Contributor Author

Yeah, I don't have a clear picture of how it would work, I give you that. But is it that niche? Does people just run fastqc once and then they don't care anymore? Don't you wanna fix the issues and then run it again to see that its fixed?

@ewels
Copy link
Member

ewels commented Sep 18, 2015

Yeah :) Can see that from the numbers though, or highlighting then looking at the plots.. But MultiQC isn't just for FastQC, so this would have to be written in such a way that any module can use it. Not so easy! (Though could restrict to the general stats table I guess)

@ewels
Copy link
Member

ewels commented Sep 21, 2015

Have a look again after the latest commit (e91133c) - no linking between samples, but I've tried to improve how quantitative the general stats is, for better sample comparison.

Let me know what you think!

@alneberg
Copy link
Contributor Author

Very pretty indeed! Gotta love how fast this project evolves! 👍
I think if I had a way to sort the rows so that individual samples ended up together, I would be satisfied, especially since I just discovered the fancy "hide samples" function. Right now I managed to do that by sorting on the MReads column, which will only work in certain cases. But it might just be enough for now. Yey!

@ewels
Copy link
Member

ewels commented Sep 21, 2015

Hah, enjoy it whilst it lasts! 😉

I could add a button above the table when highlights are created, allowing you to sort by highlights. Seems like it would be a good idea generally anyway.

@ewels
Copy link
Member

ewels commented Sep 22, 2015

ps. You know that you can manually sort the rows, right @alneberg?

alexiswl pushed a commit to alexiswl/MultiQC that referenced this issue Aug 15, 2022
DRAGEN mapping metrics support NA vlues
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants