Peter Johnson
Peter Johnson
Why can’t you just decorate using .withName()? Eg Commands.sequence(a,b,c).withName(“quick auto”)?
I assume the put() call was modeled after the Java Map put() function, which returns the previous value, not the map itself. The current code returns void, but it might...
Isn't the error message for raw loadLibrary() just "can't load native library"? I believe the reason we had the more descriptive error was for the (fairly common) case when users...
Let’s do that, yes.
The `Subsystem` name is indeed confusing for teams because they think of `Subsystem` in terms of system terminology, not command scheduler execution. It's often a 1:1 mapping, but is not...
Other than the fact I hate abseil with the flame of 1000 suns because it is the definition of dependency hell, if we have to upgrade, we have to upgrade....
> Wasn't the iteration being done via an iterator (and then the removal also being done through the iterator)? Sort of. It used a range for, which has an internal...
No. Neither does std::set.
> You've started to do this with `PubSubOption`, but I suggest having an easy way to add full metadata -- this will lay the groundwork for changing the way Shuffleboard...
Won't it be confusing to have both `publishBoolean()` and `booleanPublisher()`?