photonvision icon indicating copy to clipboard operation
photonvision copied to clipboard

Add Limelight3 hardware config

Open Craftzman7 opened this issue 2 months ago • 4 comments

Description

Closes #2091

I know the pins and the FOV are correct, unsure about the PWM range or frequency. I haven't had a chance to test this but I was able to figure this out when attempting to turn on the LEDs on a blank install of Raspberry Pi OS back in 2023. I will have access to an LL3 tomorrow and hope to test then.

Meta

Merge checklist:

  • [x] Pull Request title is short, imperative summary of proposed changes
  • [x] The description documents the what and why
  • [ ] This PR has been linted.
  • [ ] If this PR changes behavior or adds a feature, user documentation is updated
  • [ ] If this PR touches photon-serde, all messages have been regenerated and hashes have not changed unexpectedly
  • [ ] If this PR touches configuration, this is backwards compatible with settings back to v2025.3.2
  • [ ] If this PR touches pipeline settings or anything related to data exchange, the frontend typing is updated
  • [ ] If this PR addresses a bug, a regression test for it is added

Craftzman7 avatar Nov 11 '25 18:11 Craftzman7

PWM range and frequency can probably be left unspecified to use default values.

thatcomputerguy0101 avatar Nov 12 '25 18:11 thatcomputerguy0101

The current values are just yanked from the LL2+ config.

Craftzman7 avatar Nov 12 '25 18:11 Craftzman7

ledPWMFrequency is not currently parsed by PV, and ledPWMRange defaults to [0, 100]. It seems like ledPWMFrequency was forgotten in the documentation and the LL2+ config after #127 removed the option.

thatcomputerguy0101 avatar Nov 12 '25 18:11 thatcomputerguy0101

Tested, working.

Craftzman7 avatar Nov 13 '25 22:11 Craftzman7

Can you confirm that this works on the latest commit on main? #2171 changed how GPIO works.

Gold856 avatar Dec 31 '25 07:12 Gold856