photonvision Create NTLogAppender

I don't think this adds much bandwidth, and massively helps debug photon when looking at wpilogs of NT data

based on #1298

Mar 22 '24 01:03 mcm001

It's super easy to must not send debug prints to NT, if we want?

Mar 22 '24 03:03 mcm001

It's super easy to must not send debug prints to NT, if we want?

I meant more adding a UI toggle to handle logging to NT

Mar 22 '24 06:03 srimanachanta

Yeah idk. Unless teams see a legit value add from having photon logs in their wpilogs, I may punt on this one

Mar 23 '24 00:03 mcm001

Yeah idk. Unless teams see a legit value add from having photon logs in their wpilogs, I may punt on this one

I think it would be useful, especially for teams that do log

Mar 24 '24 22:03 srimanachanta

On 2nd thought, maybe the rio nt server will get bogged down. Logging rio console to wpilog via advantagekit slowed things down for us.

Instead, maybe it would be better for photonlib to publish the name of the wpilog and have photon live rename the log file to match what the wpilog is called.

May 13 '24 16:05 TheTripleV

That's decently clean. I do think we can reevaluate this come 2027 (ROSconsole, for example, is a thing and exists). Wanna make an issue to track auto-renaming logs instead? Or at the very least print in the log when valid match number is sent?

May 13 '24 18:05 mcm001