adafruit_python_pca9685 and adafruit_circuitpython_pca9685 conflict

[quebeh]

Hey guys, i want to use the newer adafruit_circuitpython_pca9685 library for other utilities. But it seems like donkeycar is using the old, deprecated library adafruit_python_pca9685.

Is there any replacement for donkeycar.parts.actuator.PWMThrottle (ESC module) that uses the new library?
I have tried to download the old one's source code, but it seems like having both the modules having its own instances of PCA9685 objects will reduce the efficiency.

Any help is appreciated
Thank you

[Ezward]

We have no plans to change that library, but we are definitely open to accepting a PR.

The old PCA9685 class is being deprecated in favor of PulseController which takes a PwmPin which can source Pwm from GPIO or PCA9685. That is what PWMSteering and PWMThrottle use to source the PWM signal. So there is no need to touch those; you want to change the implementation of the PCA9685 pwm pin at PwmPinPCA9685.

"both the modules having its own instances of PCA9685 objects will reduce the efficiency"

You are correct that the PwmPinPCA9685 implementation does use one PCA9685 instance per pin via the factory method pca9685. I don't think this is an issue but you are free to change that implementation detail if you decide to open a PR.

[quebeh]

Alright. Thank you very much. I'll get a better understanding of the pins.py code and possibly make a pull request.