search

heudiasyc / rt_of_low_high_res_event_cameras

Code for the "Real-Time Optical Flow for Vehicular Perception with Low- and High-Resolution Event Cameras" article
Other
25 stars 6 forks source link

Docker plans? #5

Open OMGAmici opened 1 week ago

OMGAmici commented 1 week ago

Amazing work here, thank you!

Are there any plans or would there be any consideration made towards compiling your code into a Docker container to streamline set up?

vbrebion commented 3 days ago

Hi,

Thank you very much for your interest in our work! And thank you for the idea, being able to compile / run the code in Docker should indeed simplify the set up for most users!

I am not an expert in Docker, but I think I have been able to create a Dockerfile which should answer the issue. I have pushed it into a new docker branch, and the README has been updated to give instructions on how to compile and run the code with Docker.

Please do not hesitate to test it and to let me know if there is any issue / if you see any improvement that could be brought. Once this is OK, I will merge https://github.com/heudiasyc/rt_of_low_high_res_event_cameras/pull/6 to add it to the main branch.

Best, Vincent

OMGAmici commented 7 hours ago

Thank you so much! This is a great addition. Just tried building the docker container and it went smoothly.

One detail that others might run into depending on the machine they're working on is that line 13 in the Dockerfile might give issues due to certificate issues. If so, a quick fix I found was replacing it with wget --no-check-certificate -qO - https://raw.githubusercontent.com/ros/rosdistro/master/ros.asc | apt-key add -