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.

Sign up for GitHub

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

Jump to bottom

plotHeatmap #153

Closed
ivokwee opened this issue May 21, 2020 · 3 comments
Closed

plotHeatmap #153

ivokwee opened this issue May 21, 2020 · 3 comments
Labels
feature-request Request to add new functionality

Comments

@ivokwee
Copy link

ivokwee commented May 21, 2020

Hi. In the deepTools module, it would be nice to include plotHeatmap inside the pipeline as the computeMatrix is already calculated. Thanks. Ivo
@drpatelh drpatelh added the feature-request Request to add new functionality label May 30, 2020
@drpatelh
Copy link
Member

Hi @ivokwee I thought this may be a bit less useful because as it stands the heatmap would only be generated for a single IP (and maybe its corresponding IP). Given the plotProfile already gives you an idea as to the "average" enrichment at the TSS and it can be combined in MultiQC I thought that may be more useful?

@ivokwee
Copy link
Author

ivokwee commented May 31, 2020

Well having both is cool. Also the kmeans clustering heatmap is very nice. But if you want the pipeline to be "minimal but complete" then maybe let the user can do it afterwards manually.

@drpatelh
Copy link
Member

drpatelh commented Jun 2, 2020

Added this in here for you @ivokwee 23cc664 Thanks for the suggestion 👍

@drpatelh drpatelh closed this as completed Jun 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature-request Request to add new functionality
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ivokwee @drpatelh