Skip to content

fix: fixes behavior with DIO#143

Merged
tylerkron merged 2 commits into
mainfrom
fix-digital-output-value
Apr 11, 2025
Merged

fix: fixes behavior with DIO#143
tylerkron merged 2 commits into
mainfrom
fix-digital-output-value

Conversation

@tylerkron
Copy link
Copy Markdown
Contributor

  • on/off on DIO Output was backwards
  • the app made a bad assumption of the data being sent for DIO. It assumed that only input would be sent. In reality, both input and output are sent. This caused a bad mapping of DIO values to channel.

Closes #141
Closes #142

…e the false assumption that the only data returned would be inputs. But in actuality it gets all active channels, including outputs.
@tylerkron tylerkron requested a review from a team as a code owner April 11, 2025 04:10
@tylerkron tylerkron merged commit 1f2c364 into main Apr 11, 2025
@tylerkron tylerkron deleted the fix-digital-output-value branch April 11, 2025 04:16
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.

Bug when using a mix of DIO inputs and outputs Digital Output On/Off toggle is reversed

1 participant