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.

Sign up for GitHub

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

Jump to bottom

Do not assume digital and analogue inputs are enabled #8

Open
dajtxx opened this issue Sep 7, 2022 · 2 comments
Open

Do not assume digital and analogue inputs are enabled #8

dajtxx opened this issue Sep 7, 2022 · 2 comments
Labels
firmware Issues and enhancements related to firmware development.

Comments

@dajtxx
Copy link
Contributor

dajtxx commented Sep 7, 2022

Don't send values for digital and analogue inputs that are not in use.
@dajtxx dajtxx added the firmware Issues and enhancements related to firmware development. label Sep 7, 2022
@HarveyBates
Copy link
Member

this for the kangaroo?

@dajtxx
Copy link
Contributor Author

dajtxx commented Nov 15, 2022

No, it was a general point made early on to say if possible the firmware shouldn't send say 4 values if there are 4 A2D ports, but only one of them is in use.

The ICT node sends values for all A2D & digital inputs and it's up to the decoder to know whether the values are useful or not.

I'm not sure how viable this is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
firmware Issues and enhancements related to firmware development.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dajtxx @HarveyBates