Skip to content

Expose STARBackend.move_channel_probe_z()#961

Open
BioCam wants to merge 6 commits intoPyLabRobot:mainfrom
BioCam:Expose-STARBackend.move_channel_probe_z()
Open

Expose STARBackend.move_channel_probe_z()#961
BioCam wants to merge 6 commits intoPyLabRobot:mainfrom
BioCam:Expose-STARBackend.move_channel_probe_z()

Conversation

@BioCam
Copy link
Collaborator

@BioCam BioCam commented Mar 25, 2026

No description provided.

BioCam added 3 commits March 25, 2026 10:09
Bypasses the C0 master module (KZ) by sending ZA directly to the pip channel, enabling Z moves with configurable speed/acceleration even when the firmware's tip-picked-up flag is incorrectly set.
Also updates `move_channel_z` docstring.
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR exposes a new low-level per-channel Z movement API on the Hamilton STAR backend, intended to bypass master-module routed Z moves when the instrument’s internal “tip picked up” state causes incorrect behavior.

Changes:

  • Clarifies move_channel_z semantics in its docstring (tip-dependent reference point).
  • Adds STARBackend.move_channel_probe_z() to move an individual channel’s probe Z-drive directly via the channel module.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

BioCam and others added 2 commits March 25, 2026 12:26
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@BioCam
Copy link
Collaborator Author

BioCam commented Mar 25, 2026

@rickwierenga, @ben-ray, @cmoscy, @jrast

Should we call this STARBackend.move_channel_stop_disc_z() and apply "stop_disc" naming for everything that refers to it?
It seems clearer to me than "probe" because very little Hamilton documentation references "probe", as opposed to "stop disc":

Screenshot 2026-01-01 at 14 52 20

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