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

Add spaceranger test reports #256

Merged
merged 1 commit into from
Jul 30, 2023
Merged

Add spaceranger test reports #256

merged 1 commit into from
Jul 30, 2023

Conversation

grst
Copy link
Contributor

@grst grst commented Jul 10, 2023

Testdata for MultiQC/MultiQC#1945

Copy link
Member

@ewels ewels left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ewels ewels merged commit 3928906 into MultiQC:master Jul 30, 2023
@ewels
Copy link
Member

ewels commented Jul 30, 2023

Wow, this is from very subsampled data 😅 I don't suppose you have any reports kicking around with more realistic values? It would help to sanity check the resulting MultiQC report a bit more..

@grst
Copy link
Contributor Author

grst commented Jul 31, 2023

This is the test data from the nf-core/spatialtranscriptomics pipeline. The only real data I have is proprietary and I don't think I'm allowed to share the report publicly.

@ewels
Copy link
Member

ewels commented Jul 31, 2023

Is there any example data on the 10x website? What about full size tests for the pipeline?

@grst
Copy link
Contributor Author

grst commented Jul 31, 2023

There are some samples on the 10x website, but afaik it's just single unrelated samples, so they would make a boring QC report.

Full-sized test dataset is an open issue: nf-core/spatialvi#48.
The proper solution would be to find a suitable full-sized test-dataset.

@ewels
Copy link
Member

ewels commented Jul 31, 2023

Can have multiple HTML reports with one sample though, this would be in addition to what you already have. So I think that might still be worth it?

Can add the full size test results later maybe as well

@grst
Copy link
Contributor Author

grst commented Aug 2, 2023

See #260

I was first afraid I'd have to run the pipeline on the raw data, but the HTML reports are directly available from the 10x website.

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

Successfully merging this pull request may close these issues.

2 participants