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

Color key and density info legend #50

Open
juferban opened this issue Jan 30, 2016 · 15 comments
Open

Color key and density info legend #50

juferban opened this issue Jan 30, 2016 · 15 comments

Comments

@juferban
Copy link

Hi,
Great job with the d3heatmap library. I really like it.
Is there any plan to add a color key and density info legend box similar to the one in the picture?
In heatmap2 it is generated using density.info and it is very useful to better understand the data in being plotted.

colorkeyhist

Thanks a lot,

Julio

@bextra
Copy link

bextra commented Mar 6, 2016

I would be interested in this functionality as well.

@pierduemila
Copy link

Likewise, key=TRUE does't seem to work.

@alanocallaghan
Copy link

This repository doesn't seem to be particularly active at the moment. I have implemented a primitive version of this in my fork. I'll try to polish it over time

https://github.com/Alanocallaghan/d3heatmap

@talgalili
Copy link
Owner

Hello @alanocallaghan and everyone.
As you have noticed, Joe seems to be busy with other projects.
I have currently moved my efforts to the new heatmaply package:
https://github.com/talgalili/heatmaply
It is focused on combining ggplot2 with plotly to produce cluster heatmaps, and is currently offering more options than d3heatmap. It also comes with a color key legend built in. I was not yet able to implement density info (this would require more work on plotly::subplot by the people working on it).
The package is still young, so if anyone has feature or pull requests - I will be happy to address them.

With regards,
Tal

@juferban
Copy link
Author

Hello @alanocallaghan,

Could you provide some example on how to generate the color key with your code? I check your repository and even though I can see the jsfiddle example with the color key on it I was not able to figure out what is the parameter to add into d3heatmap.

Thanks a lot.

@alanocallaghan
Copy link

Hi @juferban,
Thanks for the heads up. I had it set as the default (I think not using a colour key is rather silly). I've now updated it to use the logical parameter show_color_legend. I've also tweaked the CSS, updated the documentation and fixed a minor bug so I would recommend installing the latest version.

Cheers,
Alan

@bextra
Copy link

bextra commented Aug 22, 2016

@alanocallaghan i was just getting ready to do the update but saw that the most recent commit on this git repo was a couple months ago. Should we be installing from here or the CRAN directly?

@talgalili
Copy link
Owner

Hi @kbeck527 - I was wondering, is there currently a reason to prefer d3heatmap over heatmaply?

@bextra
Copy link

bextra commented Aug 22, 2016

@talgalili Just trying to make sure legacy code is the best it can be. We are migrating to heatmaply but it isn't implemented for all of our visualizations yet.

@talgalili
Copy link
Owner

I understand - thanks for the explanation :)

@bextra
Copy link

bextra commented Aug 22, 2016

Update: @alanocallaghan the CRAN version is 0.6.1.1 and dated February. So where would this update be?

@alanocallaghan
Copy link

Hi @kbeck527, I was referring to the recent commits on my branch (https://github.com/Alanocallaghan/d3heatmap). You can find instructions on there for installing that particular branch and version. I'm not a contributor to this particular repository as it doesn't seem to be active at the moment

@bextra
Copy link

bextra commented Aug 22, 2016

Ah I see. Thank you @alanocallaghan. Very helpful changes. Are you planning to do a pull request in case it does become active again?

@alanocallaghan
Copy link

If I find the time, absolutely

@juferban
Copy link
Author

juferban commented Aug 22, 2016

Hi Alan @alanocallaghan

Thanks a lot for the information and the update. I agree that there may be no reason to not have the color legend enabled but my first try on the code was not showing the legend so I thought I was missing something. Instead of the legend I still get what it looks like a couple of letters showing on the top-left corner. I am reviewing my code to see if there may be something interfering with your library and will let you know of any issues I may encounter.
Once again thanks a lot for your help and for sharing your code.

Julio

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

5 participants