search

gtagency / buzzmobile

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

Prune repo of unused things #39

Closed irapha closed 7 years ago

irapha commented 7 years ago

Such as two of the image_const nodes (we only need one, and it should arguably live in tools). Or even the lane_detector, which we are scraping for the future road_detector node aka agencynet

joshuamorton commented 7 years ago

Ehh, I agree with the first part of this (redundant stuff), but the whole point of buzzmobile was to potentially have various ways of doing things, and use it as a testbed for stuff.

irapha commented 7 years ago

I'd argue those can happen in branches but master should be cleaner

joshuamorton commented 7 years ago

Depends, there's value in being able to test multiple things at once, though I guess if you're smart enough with git you can merge non-branches into each other to get that effect. Worth weighing in mind in any case.

On Wed, Oct 19, 2016 at 12:44 AM Raphael Gontijo Lopes < notifications@github.com> wrote:

I'd argue those can happen in branches but master should be cleaner

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/gtagency/buzzmobile/issues/39#issuecomment-254709147, or mute the thread https://github.com/notifications/unsubscribe-auth/AEz4AFessdkNetlfTpRwU0Y1NNmfeXzMks5q1aA1gaJpZM4KahZ3 .