Skip to content

Conversation

jasondaming
Copy link
Member

@jasondaming jasondaming commented Oct 8, 2025

Summary

Adds seealso boxes linking to vendor documentation for Pneumatic Control Module (CTRE) and Pneumatic Hub (REV) configuration and firmware updates.

Changes

  • wiring-pneumatics-pcm.rst: Added seealso box with two CTRE documentation links:
    • CAN Bus Setup guide (device IDs, firmware updates, troubleshooting)
    • PCM Bring-Up guide (testing solenoids and compressor operation)
  • wiring-pneumatics-ph.rst: Added seealso box linking to REV Pneumatic Hub documentation for configuration via REV Hardware Client and firmware updates

These links provide teams with vendor-specific configuration guidance that complements the wiring instructions in the main docs.

Fixes #2272

Adds seealso boxes linking to CTRE PCM documentation and REV Pneumatic Hub documentation for configuration, firmware updates, and detailed technical specifications.

Fixes wpilibsuite#2272
@sciencewhiz
Copy link
Collaborator

One thing that stopped me in the past from doing this is vendor documentation required jumping to several disparate areas. Is it better now? (I'm on mobile)

@jasondaming
Copy link
Member Author

The CTRE link was broken and still isn't great. The REV link goes to the overview and has other articles right behind it. I think this is helpful

@sciencewhiz
Copy link
Collaborator

I think at a minimum CTRE needs both https://v5.docs.ctr-electronics.com/en/stable/ch08_BringUpCAN.html and https://v5.docs.ctr-electronics.com/en/stable/ch09_BringUpPCM.html but there's also a lot of irrelevant stuff there too. Also, that doesn't have any technical specs like the PR says it will.

Changes:
- Removed claim about 'technical specifications' (not available in CTRE docs)
- Added both CAN Bus Setup and PCM Bring-Up links as suggested by reviewer
- Added brief descriptions for each link explaining what they cover
- More accurate representation of what the CTRE documentation provides

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
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

Successfully merging this pull request may close these issues.

Configuring PCM/PH instructions

2 participants