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.

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

Add Axis Offset Compensation #44

Merged
merged 2 commits into from
Dec 30, 2024
Merged

Add Axis Offset Compensation #44

merged 2 commits into from
Dec 30, 2024

Conversation

brian-eng
Copy link
Contributor

It's pretty simple to do I think, so I would recommend at least giving it a shot if you can

Originally posted by @purepani in #39 (comment)

Adding these properties as it is needed when enabling temperature compensation.

Temperature compensation disabled, no offsets

X: -43.5 uT
Y: -47.1 uT
Z: 13.068 uT

Temperature compensation enabled, no offsets

X: 8973.9 uT
Y: 8969.4 uT
Z: 14551.944 uT

Temperature compensation enabled, 0x8000 offset for all axes

X: -39.3 uT
Y: -42.6 uT
Z: 12.1 uT

Required when temperature compensation is enabled
@purepani
Copy link
Contributor

This is definitely a good first step. Could you squash the commits?

Also, in the future, maybe someone can make an API to make it a bit more intuitive to set these values, rather than setting the byte values directly, but that would require a bit more design and thought.

@brian-eng
Copy link
Contributor Author

This is definitely a good first step. Could you squash the commits?

Sure if I need to, figured someone would just do the squash on the merge though.

Copy link
Contributor

@FoamyGuy FoamyGuy left a comment

Choose a reason for hiding this comment

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

Looks good to me. I tested on a Feather S3 TFT 9.2.1 and found behavior matching what is mentioned int he original comment on this PR.

@FoamyGuy
Copy link
Contributor

It appears that some configuration has disabled the squash and merge functionality.
image

There are only 2 commits here anyhow, I think it's fine to merge them as-is.

@FoamyGuy FoamyGuy merged commit 7762818 into adafruit:main Dec 30, 2024
1 check passed
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Dec 31, 2024
Updating https://github.com/adafruit/Adafruit_CircuitPython_MLX90393 to 2.3.0 from 2.2.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_MLX90393#44 from brian-eng/offsets

Updating https://github.com/adafruit/Adafruit_CircuitPython_MPU6050 to 1.2.6 from 1.2.5:
  > Merge pull request adafruit/Adafruit_CircuitPython_MPU6050#38 from FoamyGuy/samd21_ram_insufficient

Updating https://github.com/adafruit/Adafruit_CircuitPython_OV5640 to 1.2.2 from 1.2.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_OV5640#33 from FoamyGuy/ev_whitebalance_getter_fix

Updating https://github.com/adafruit/Adafruit_CircuitPython_LED_Animation to 2.11.0 from 2.10.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_LED_Animation#123 from FoamyGuy/period_property_more_animations

Updating https://github.com/adafruit/Adafruit_CircuitPython_Wiz to 1.1.0 from 1.0.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_Wiz#4 from FoamyGuy/scanning_and_mac_map

Updating https://github.com/adafruit/Adafruit_CircuitPython_Bundle/circuitpython_library_list.md to NA from NA:
  > Updated download stats for the libraries
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.

3 participants