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

Potentially removing colormap #74

Closed
muschellij2 opened this issue Mar 15, 2019 · 3 comments
Closed

Potentially removing colormap #74

muschellij2 opened this issue Mar 15, 2019 · 3 comments

Comments

@muschellij2
Copy link
Collaborator

So colormap requires V8 which requires a 3rd party install of v8 on machine. Probably going to cause some headaches downstream for installers. Maybe use the viridis package if wanting that palette? https://cran.r-project.org/web/packages/viridis/vignettes/intro-to-viridis.html. Or you can use colorRampPalette or colorRamp.

@muschellij2
Copy link
Collaborator Author

@stnava
Copy link
Member

stnava commented Mar 16, 2019

we had viridis as a package dependency for a long time ( in ANTsR )

colormap advantages: one line to have access to a bunch of different useful common color mapping schemes

disadvantages: dependencies, does not have basic maps like red, blue

anyway, those are my comments. i would suggest a fork that works on the various issues associated wtih plot.antsImage many of which go beyond this. it should be rewritten, decluttered.

@stnava
Copy link
Member

stnava commented Jun 18, 2019

Closed by removing these dependencies.

@stnava stnava closed this as completed Jun 18, 2019
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

2 participants