Skip to content
This repository has been archived by the owner. It is now read-only.

Scatterplot & some fixes #6

Merged
merged 15 commits into from
Feb 13, 2018
Merged

Conversation

HendrikSchultheis
Copy link
Contributor

Added categorical scatterplots. If the scatterplot receives non numeric values for the color-axis it will map distinct colors to each value rather than using a color gradient. Also a warning is shown for more than 10 categories.

patched:

  • DT's dataTableProxy call changed in version 0.3 so that it no longer needs id wrapped in session$ns -> deleted respective session$ns and added dependency >= 0.3 for wilson
  • columnSelector no longer returns logical(0) on init

@HendrikSchultheis
Copy link
Contributor Author

HendrikSchultheis commented Jan 30, 2018

I also noticed a problem with heatmaply and tracked it down to be a version problem aswell as with DT.

Note: Hovertexts are not working with this version. See here. Hopefully this will be fixed soon.

@HendrikSchultheis HendrikSchultheis changed the title Scatterplot Scatterplot & some fixes Jan 30, 2018
@HendrikSchultheis
Copy link
Contributor Author

Regarding heatmaply I found a few more changes and applied them to wilson. And furthermore added some more warnings/ checks for insufficient plot data.

@HendrikSchultheis
Copy link
Contributor Author

With heatmaply version 0.14.1 Hovertexts are working again. Should the dependency be updated to this version?

@jenzopr jenzopr merged commit 9492c1b into loosolab:master Feb 13, 2018
@HendrikSchultheis HendrikSchultheis deleted the scatterplot branch April 3, 2018 13:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants