search

gtagency / buzzmobile

An autonomous parade float/vehicle
MIT License
18 stars 3 forks source link

Some launch files and memory leak fixes #71

Closed kpberry closed 8 years ago

kpberry commented 8 years ago

Fixes #72 #62 #61 @kpberry did I miss any issues?

joshuamorton commented 8 years ago

drat I missed this, does this include all python dependencies, I'm not seeing the cv2, matplotlib, or tensorflow bindings, or are those done elsewhere, or requests, which is missing.

kpberry commented 8 years ago

It doesn't, but I should be able to add them in a few minutes.

joshuamorton commented 8 years ago

Its also similarly likely that we'll almost immediately need to pull everything out into a build.sh file to do the tf, matplot, and cv2 stuff, but that's fine, better to have it all in one place.

irapha commented 8 years ago

Cv3 has to be installed in a special way. The details as to why are in an issue you can dig up if you want.

Neither matplotlib not tensorflow are currently dependencies in Buzzmobile.

On Wed, Oct 26, 2016, 00:47 Josh Morton notifications@github.com wrote:

drat I missed this, does this include all python dependencies, I'm not seeing the cv2, matplotlib, or tensorflow bindings, or are those done elsewhere?

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/gtagency/buzzmobile/pull/71#issuecomment-256248517, or mute the thread https://github.com/notifications/unsubscribe-auth/AHC0h0v4fjJxmntg2sPJ_0sCR2Ms4BEKks5q3ttVgaJpZM4KgpR4 .

irapha commented 8 years ago

Idk why we would depend on matplotlib tbh. But yes there's an open issue for standardizing installation.

On Wed, Oct 26, 2016, 00:56 Josh Morton notifications@github.com wrote:

Its also similarly likely that we'll almost immediately need to pull everything out into a build.sh file to do the tf, matplot, and cv2 stuff, but that's fine, better to have it all in one place.

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/gtagency/buzzmobile/pull/71#issuecomment-256249581, or mute the thread https://github.com/notifications/unsubscribe-auth/AHC0h8I84awbJPznwbbApxzpgb9S6Kljks5q3t1kgaJpZM4KgpR4 .