HX71708 Sensor Support#895
Open
garethky wants to merge 2 commits into
Open
Conversation
garethky
force-pushed
the
pr-hx71708-sensor
branch
from
June 22, 2026 17:52
06f6930 to
09191e1
Compare
garethky
force-pushed
the
pr-hx71708-sensor
branch
from
July 7, 2026 04:14
09191e1 to
216002f
Compare
2 tasks
Contributor
|
CI is green after the branch update. Two asks plus a cleanup on this one:
|
rogerlz
added a commit
that referenced
this pull request
Jul 14, 2026
The unbuffer wrapper masks the docker build exit status. On the PR #895 run the image rebuild failed with the ar100 size overflow, the step still reported success, and the klippy tests silently ran against the stale docker hub image with outdated dicts, failing with a confusing KeyError instead of the real build error. Drop unbuffer and the expect install. With --progress=plain the output pipes fine through sed and the exit status now propagates through pipefail. Reproduced locally with a failing Dockerfile. Signed-off-by: Rogerio Goncalves <rogerlz@gmail.com>
garethky
force-pushed
the
pr-hx71708-sensor
branch
from
July 14, 2026 19:11
5643d28 to
741b34a
Compare
added 2 commits
July 14, 2026 12:21
Ella Fox - ella@fox.gal Signed-off-by: Gareth Farrington <gareth@waves.ky>
* Put the sub classes in charge of reading the config and deciding how many additional clock pulses are issued to configure the chip. * Fix gain options * Doc Updates Signed-off-by: Gareth Farrington <gareth@waves.ky>
garethky
force-pushed
the
pr-hx71708-sensor
branch
from
July 14, 2026 19:30
741b34a to
2338b7e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is the code to integrate the HX71708 variant of the HX71x sensor family. This has been initially developed and tested out by @EllaFoxo.
There is a refactor of the of the MCU command so that you are commanding a sequence of bits that will be sent at the end of each sampling loop.
Checklist