Implement NetworkAlerts

[TheGamer1002]

See here

[TheGamer1002]

I'm un-drafting this.

[thenetworkgrinch]

I like these changes but I am hesitant to force users to use AdvantageScope (even though it is great!) Does this fallback to reporting via DriverStation?

[thenetworkgrinch]

I'd be worried if the setText function isnt used when the alert is activated.

[TheGamer1002]

I like these changes but I am hesitant to force users to use AdvantageScope (even though it is great!) Does this fallback to reporting via DriverStation?

It does indeed! It actually prints to the DriverStation even if the plugin isn't used.

[TheGamer1002]

I'd be worried if the setText function isnt used when the alert is activated.

When it's activated, updating the text re-prints it and sends the updated text to the network table.

[Technologyman00]

I think it looks great!

[thenetworkgrinch]

@TheGamer1002 This merge may be removed in the future. Upon further testing the memory allocation of the Alert class is absolutely insane (greater than 50% of all YAGSL utilization right now). If you would like to save it, you may want to optimize it.

[thenetworkgrinch]

[TheGamer1002]

Oh jeez, that is... a LOT of memory. If you want to remove it as a temporary fix, I can see what can be done about this, but this should definitely not be happening.

[thenetworkgrinch]

@TheGamer1002 I will remove it right before week 0 comp if you don't submit a patch by then.

[TheGamer1002]

I think I fixed it, see the YAGSL PR