support digital, single analog, and dual analog pressure sensors

[gcschmit]

The pneumatics subsystem currently only supports dual analog pressure sensors (from Rev). It would be relatively straight forward to add a parameter to the constructor to select between the above three options. (If using a digital pressure sensor, replace enableCompressorAnalog with enableCompressorDigital.)

[gcschmit]

From CD message: "There is a pneumatics.getCompressorConfigType() method that I believe tells you if the PDH is configured for digital or analog pressure sensor."

[randomstring]

I spent some time look at this today. Just detecting the compressor type requires the compressor mode to be set prior. I think the better option is to select the compressor mode when instantiating the pneumatic class. See my (yet untested):
FRC-Sonic-Squirrels@509db00

There is also a hybrid mode, that uses both digital and analog sensors, so any calls to enableCompressorAnalog() could be replaced with enableCompressorHybrid() and I think it should work with either or both.
https://first.wpi.edu/wpilib/allwpilib/docs/release/java/edu/wpi/first/wpilibj/PneumaticHub.html#enableCompressorHybrid(double,double)

[gcschmit]

Maybe an enumeration with more options:

• single digital
• single analog
• hybrid (1 digital; 1 analog)
• double analog
• double hybrid (1 digital; 2 analog)

or maybe two parameters – the first specifies the enumerated value related to compressor control and the second a boolean to indicate if a second analog pressure sensor is present to log pressure downstream of the regulator.

• single digital
• single analog
• hybrid (1 digital; 1 analog)

[randomstring]

I got my version of Pneumatics working with the digital sensor on my fork. Looks very similar to your solution.