-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add customizable color scale for visualization #18
Add customizable color scale for visualization #18
Conversation
Is red to green a good choice? These colors are commonly difficult to distinguish for color blind people. |
You are right, but I don't want to change the default, just give another option. Anyway, this PR will be changed based on the discussion here ros-industrial/reach#58 |
I was going to post something similar. Most flexible option would be to allow configuring colour scales. But that might require more work. |
Any ideas why this is happening on the rolling build? It seems to happen when we link the reach study node to the library being built by this repo. I'm not sure why some distributed library that is not being built in this repo is complaining about an undefined reference |
Turns out the |
d5dce96
to
f8c7a70
Compare
See ros-industrial/reach#58
edit: building of CI fails as the mentioned PR is required as a basis