Re: Position and velocity PIDControllers using same encoder
 

Again, it's a matter of weighing the ugly, but perhaps you could free the encoder you aren't using (if I understand correctly, setting the variable(s) referencing it to null and possibly running garbage collection will accomplish this), then initiate the new one.

Re: Position and velocity PIDControllers using same encoder
 

If I were never going to use them at the same time, this would probably work - but, as mentioned earlier in the thread, the notion is to run a cascading control loop on a turret where the position controller outputs to a velocity controller using the same encoder.

Re: Position and velocity PIDControllers using same encoder
 

Doing this between two PIDController Notifiers, which are real-time processes, isn't a good idea since the garbage collector introduces a lot of non-determinism and delays. If they are running at low enough sample rates, it might work, but there's no guarantee. Forcing the garbage collector to run would also collect any other garbage lying around the Notifiers don't care about.

I thought of another option that avoids needing to do any trickery with multiple Encoder objects. You could inherit from the PIDController class with two classes called PositionPIDController and VelocityPIDController. They would both override calculate(), call m_pidInput.setPIDSourceType() with the PIDSourceType they each want to use, then call the base class calculate().

As a side note, you may need to synchronize access to the Encoder instance in calculate() since the PIDController instances are sharing it.

Re: Position and velocity PIDControllers using same encoder
 

Here's a wrapper that 2485 wrote that does this.

https://github.com/team2485/frc-2016...dDistance.java