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

Do we need multiQC? #74

Closed
jpfeuffer opened this issue Feb 17, 2022 · 5 comments
Closed

Do we need multiQC? #74

jpfeuffer opened this issue Feb 17, 2022 · 5 comments
Labels
enhancement New feature or request

Comments

@jpfeuffer
Copy link
Collaborator

Description of feature

We already have pmultiQC. Do we need multiQC to run pmultiQC or something?
If not, remove it.

@jpfeuffer jpfeuffer added the enhancement New feature or request label Feb 17, 2022
@jpfeuffer
Copy link
Collaborator Author

@daichengxin the idea would be to only have pMultiQC and send an email with the report, like multiqc does.

@daichengxin
Copy link
Collaborator

Good idea. I am trying to inplement these functions in pmultiqc

@jpfeuffer
Copy link
Collaborator Author

I don't think you need to implement it in pMultiQC but you need to use pMultiQC in the place of multiQC and set the nextflow variables that define the report to be be emailed to the output of pMultiQC instead

@daichengxin
Copy link
Collaborator

Yes. I am sending a example in slack, and removing multiqc.

@daichengxin
Copy link
Collaborator

done #82

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants