-
Notifications
You must be signed in to change notification settings - Fork 5
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
Move to WG repo? #5
Comments
It is also used by the RobotWebTools packages. Why do you think the WG repo would be a better fit? |
I'm not aware of anything in RWT that uses this a.t.m. We waiting to here if we should switch over (e.g., for something like https://github.com/RobotWebTools/mjpegcanvasjs/tree/devel) but were advised/decided to use http://ros.org/wiki/mjpeg_server until better RTC came along. The massive discussion on moving packages comes from the attept to keep RWT a minimul set of standard tools/libraries (e.g., https://github.com/RobotWebTools/roslibjs or https://github.com/RobotWebTools/rosbridge_suite) and to move specific implementations (e.g., https://github.com/RobotWebTools/nav2djs) into respective repos. This also prevents the RWT github from becoming a "best friends" club. An index will also be started as well to list all projects/repos. |
After discussing on Friday, we're looking at keeping this in Robot Web Tools as it's needed for an upcoming feature in ros3djs. We did discuss renaming ros_http_video_streamer. One idea I said was webm_video_streamer. However, after discussing with @jkammerl, it's possible to add h264 encoding in the future (instead of just webm, like it is currently), which would allow us to support a wider range of browsers and devices (like an ipad). |
|
I guess you would like the package name to be shorter, right? What about ros_web_video? |
sounds good to me |
Agreed, ros_web_video sounds good. |
I renamed the package in my fork: https://github.com/jkammerl/ros_web_video Since we are already creating debs from it, I am not sure what the best way is to deprecate the current package. Maybe we should just create a new package and deprecate the ros_http_video_streamer in hydro. What do you think? |
Not sure where this should live -- @baalexander ?
The text was updated successfully, but these errors were encountered: