Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature request: Documentation: Full TalonFX Current Limit API guide #63

Open
samneff72 opened this issue Jan 27, 2020 · 3 comments
Open

Comments

@samneff72
Copy link

Feature Requests

Feature request in as much detail as possible.

Update "New API in 2020" section at https://phoenix-documentation.readthedocs.io/en/latest/ch13_MC.html#new-api-in-2020 to include complete details (descriptions & typical values) associated with common settings available in the TalonFX Current Limiting API, or at least direct links to code examples that work for supply-limiting in 2020.

Stretch goal: complete details for all settings for TalonFX Current Limiting on both supply & stator sides

Why it is beneficial.

The existing (1/27/2020) public-facing current limiting documentation does not have a complete list of settings & common values required to make the feature work in 2020 on the TalonFX API.

https://phoenix-documentation.readthedocs.io/en/latest/ch13_MC.html#new-api-in-2020

The existing public-facing current limiting code examples are for the 2019 API

https://github.com/CrossTheRoadElec/Phoenix-Examples-Languages/tree/master/Java/Current%20Limit/src/main/java/frc/robot

https://github.com/CrossTheRoadElec/Phoenix-Examples-Languages/tree/master/Java/CurrentClosedLoop/src/main/java/frc/robot

The ability to limit current on the TalonFX (similar to the featureset available on the TalonSRX) was a significant part of our team's buying decision on the Falcon500. If the feature is available but is not documented for us to activate, it might as well not be available.

A series of steps to test the feature after it is implemented

Documentation update

@samneff72
Copy link
Author

@JCaporuscio
Copy link
Member

The Supply Current Limit for Talon FX is identical to the traditional current limiting in the Talon SRX.

The only difference is that the parameters are bundled into a single configuration object so only a single API call is needed.

For Java:
API Call: http://www.ctr-electronics.com/downloads/api/java/html/classcom_1_1ctre_1_1phoenix_1_1motorcontrol_1_1can_1_1_talon_f_x.html#a302c7824cf2728ee03f1b739a14e2a35
Config Object:http://www.ctr-electronics.com/downloads/api/java/html/classcom_1_1ctre_1_1phoenix_1_1motorcontrol_1_1_supply_current_limit_configuration.html

These API docs are also available in the intellisense (or javadoc jar from the installer/maven) in vscode.

These functions are listed in the doc, but I can add a code snippet of the API call.
Beyond this, is there any other point of confusion with the new API?

@samneff72
Copy link
Author

samneff72 commented Jan 27, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants