-
Notifications
You must be signed in to change notification settings - Fork 166
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
[jsk_rqt_plugins/HistogramPlot] modify to correctly run on kinetic #688
Conversation
@iroy do you men we should revert this? if so please add +1 to #704, and also please try to write the test code so that Travis find this error. I think just to write test script that import this file would be ok. @knorth55 please rewrite the code with system matlab version for now. and if you can find the code to support both system (apt) version and pip version, then create new pull requests. |
or install matplotlib |
problem in #689? any way please look at
https://github.com/ros-visualization/rqt_plot/blob/master/src/rqt_plot/data_plot/mat_data_plot.py#L54-L89
--
◉ Kei Okada
2018年9月4日(火) 13:31 Shingo Kitagawa <notifications@github.com>:
… or install matplotlib 2.0.0 with catkin_virtualenv.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#688 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAeG3MGsxhyuRsGRYd8gUsdCITDOxUbRks5uXgIjgaJpZM4SrekD>
.
|
@k-okada thank you, I found the solution. |
Kinetic
usesqt5
, andqt5
has different module structure.I refactor codes and update to use correct module for kinetic.
You can check with