Skip to content

✨(minor) Add AS5600 magnetic rotary encoder#36

Merged
kammce merged 12 commits into
libhal:mainfrom
MaliaLabor:as5600
May 24, 2026
Merged

✨(minor) Add AS5600 magnetic rotary encoder#36
kammce merged 12 commits into
libhal:mainfrom
MaliaLabor:as5600

Conversation

@MaliaLabor
Copy link
Copy Markdown
Member

No description provided.

@MaliaLabor MaliaLabor requested a review from kammce May 21, 2026 00:07
Copy link
Copy Markdown
Member

@kammce kammce left a comment

Choose a reason for hiding this comment

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

Looking really good. With these changes we can merge this in.

Comment thread demos/CMakeLists.txt
Comment thread include/libhal-sensor/as5600.hpp Outdated
Comment thread include/libhal-sensor/as5600.hpp
Comment thread include/libhal-sensor/as5600.hpp Outdated
Comment thread include/libhal-sensor/as5600.hpp
Comment thread src/as5600.cpp Outdated
Comment thread src/as5600.cpp Outdated
Comment thread src/as5600.cpp Outdated
Comment thread CMakeLists.txt
Comment thread demos/CMakeLists.txt
@kammce kammce changed the title ✨Add AS5600 magnetic rotary encoder ✨(minor)Add AS5600 magnetic rotary encoder May 21, 2026
@kammce kammce changed the title ✨(minor)Add AS5600 magnetic rotary encoder ✨(minor) Add AS5600 magnetic rotary encoder May 21, 2026
@MaliaLabor MaliaLabor requested a review from kammce May 22, 2026 23:25
Copy link
Copy Markdown
Member

@kammce kammce left a comment

Choose a reason for hiding this comment

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

Looks great. We need a few more API doc changes and we should be ready to merge.

Comment thread include/libhal-sensor/as5600.hpp
Comment thread include/libhal-sensor/as5600.hpp
@MaliaLabor
Copy link
Copy Markdown
Member Author

Deferring CI checks because they are old and need to be upgraded.

See issue #38

@kammce kammce marked this pull request as ready for review May 24, 2026 19:44
kammce added 2 commits May 24, 2026 13:04
Updated power mode enum values for clarity and added detailed explanation for angle hysteresis.
Updated comments for clarity and formatting consistency.
@kammce kammce merged commit 809d76e into libhal:main May 24, 2026
8 of 12 checks passed
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