Skip to content

Shearwater: Map Perdix 3 hardware identifiers - #123

Merged
mikeller merged 2 commits into
subsurface:Subsurface-DS9from
mikeller:feat/shearwater-perdix3-#60
Sep 1, 2026
Merged

Shearwater: Map Perdix 3 hardware identifiers#123
mikeller merged 2 commits into
subsurface:Subsurface-DS9from
mikeller:feat/shearwater-perdix3-#60

Conversation

@mikeller

Copy link
Copy Markdown
Member

Map hardware identifiers 0x39C2 and 0x4AB1 to the existing PERDIX3 model so downloads report the correct device. Existing descriptor and protocol handling already support the model.

Map hardware identifiers 0x39C2 and 0x4AB1 to the existing PERDIX3 model so downloads report the correct device. Existing descriptor and protocol handling already support the model.

Signed-off-by: Michael Keller <github@ike.ch>
Copilot AI lite review requested due to automatic review settings August 30, 2026 08:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Map the identified Petrel 3 and Teric hardware values to their existing model constants. Descriptor, transport, parser, and model-number behaviour remains unchanged.

Signed-off-by: Michael Keller <github@ike.ch>
Copilot AI review requested due to automatic review settings August 30, 2026 08:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@mikeller
mikeller merged commit 6b3b483 into subsurface:Subsurface-DS9 Sep 1, 2026
10 checks passed
@mikeller
mikeller deleted the feat/shearwater-perdix3-#60 branch September 1, 2026 08:15
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.

2 participants