Thad House
Thad House
> That's been fixed by #5838 and will be included in the next release. It’s only fixed on the allwpilib side. It still needs to be fixed on the vscode...
With the addition of `in` this should be something looked at even more. I can see in delegates being common for performance heavy code, and this could be really helpful.
`public delegate void InAction(in T value);` A delegate with some of the parameters being `in` parameters.
@HaloFour from testing, the dispatch has gotten much better in recent .NET Core versions, and if static delegates ever get added they'll be even better. And the allocation can be...
Closed by #6398
Still needs C++, Tests and Sim support. Very WIP.
> What's the difference between AnalogPotentiometer and this version of AnalogEncoder They're basically the same thing at this point. We'd probably deprecate AnalogPotentiometer since actual potentiometers are rarely used anymore....
> Maybe extract a base `AnalogSensor` class that accepts a `U(volt)` conversion function and handles offset+scaling (likely by using the above function) and then it can be used for unit-aware...