search

uvic-auvic / polaris

2019 Submarine Code.
0 stars 6 forks source link

Added Python refactor ADR #21

Closed AntonNiko closed 4 years ago

AntonNiko commented 4 years ago

This is a preliminary ADR that evaluates a transition from C++ to Python 3 for the vision node.

robwasmann commented 4 years ago

Kinetic doesn't support Python 3. Also, I'm worried about Python crashing in more unexpected ways. Since it's only compiled as it's interpreted, it won't catch errors until that particular part of the code (a particular function, say) gets executed. Personally I don't like languages like Python for anything serious where you need stability. We really don't want to vision node to crap out at an important moment.

AntonNiko commented 4 years ago

Bummer that Python 3 isn't supported in Kinetic. It sounds like ensuring that we have something working in C++ first we can use as a benchmark works better than doing Python straight out when we're not sure in the first place.

robwasmann commented 4 years ago

New members are always free to experiment with OpenCV in Python in the Vision testing repo, as well they can implement something first in Python and then convert it to C++.