Online bagger

[zrzz-hq]

The first C++ version of the online bagger has been completed and is ready for testing on a vechicle. When would be a good time to test it?

[cbrxyz]

Hey, thanks! Could you please add some instructions on running this?

[zrzz-hq]

You can type online_bagger help to display the help text. Also I'll add a README file later.

[DaniParr]

In general, the code looks good, but when you make future PRs could you provide screenshots / gifs / video links showing how to run your code and what it looks like to run the code.

After you show that it works I can rereview the code!

[zrzz-hq]

I remember we tested this on subjugator during this summer and it worked well. Carlos said he would approve the pull request but he hasn't.

[wingdeans]

I'm not sure if we need an online bagger right now. Even if we did, the extra features (TUI, progress feedback, env vars?) make the program too large for what it should do. If you left, it would be hard to find a maintainer for this.

I'll bring it up at testing but I don't think we'll move forward with this.

[zrzz-hq]

The TUI is an extra feature. Progress feedback and env vars are not, they are parts of the original version.

[zrzz-hq]

I also suggest doing issue clean-ups regularly to remove unused or obsolete issues. Letting people spend time on tools that won’t be used in the future isn’t a good use of effort.

[wingdeans]

I also suggest doing issue clean-ups regularly to remove unused or obsolete issues. Letting people spend time on tools that won’t be used in the future isn’t a good use of effort.

Agreed..