Skip to content

Add Descriptors#28

Merged
solara404 merged 5 commits into
mainfrom
add_descriptos_27
Mar 25, 2026
Merged

Add Descriptors#28
solara404 merged 5 commits into
mainfrom
add_descriptos_27

Conversation

@solara404

Copy link
Copy Markdown
Contributor

Fix: #27

@aaumar25 aaumar25 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Lets work out together the description before merging it.

Comment thread src/Config.zig Outdated
Comment thread src/Config.zig Outdated
Comment thread src/Config.zig Outdated
Comment thread src/Config.zig Outdated
Comment thread src/Config.zig Outdated
@solara404 solara404 requested a review from aaumar25 March 25, 2026 01:29
@solara404 solara404 merged commit b690589 into main Mar 25, 2026
2 checks passed
@solara404 solara404 deleted the add_descriptos_27 branch March 25, 2026 01:34
@aaumar25

Copy link
Copy Markdown
Member

I think now I understand what is hall_sensor.position.on is. It describe the carrier position relative to zero position when triggering the alarm state of the specified hall sensor.
forward: Carrier position when a carrier is moving forward and trigger alarm state of the hall sensor.
backward: Carrier position when a carrier is moving backward and trigger alarm state of the hall sensor.

I will confirm this behavior in the simulation first and update this state by myself later.

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.

Add Descriptos

2 participants