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

Restore Pos conversion to mcDEFAULT homing mode #5

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

AndrewMusser
Copy link
Contributor

This PR modifies AxisLib's reference FUB to handle restore pos homing modes differently. The general idea is to leverage more of Mapp Motion's 'out-of-the-box' capabilities with Restore Pos homing:

  • It still requires that the RestorePosition PV be passed in, but it commands a home default for axes that have a non-zero restore pos PV.
  • This means that the RestorePosition PV must also be linked in the drive's configuration for this to work.
  • The reason it is still passed in to AxisLib is so that we can check whether or not the data is valid.

@Joshpolansky
Copy link
Member

@AndrewMusser Do you remember anything about this?

@AndrewMusser
Copy link
Contributor Author

@AndrewMusser Do you remember anything about this?

@Joshpolansky Yes, vaguely. I think we did this because Mapp Motion handles the restore pos in an automated way for you when you map a PV in the drive's configuration?

@Joshpolansky
Copy link
Member

@AndrewMusser Do you remember anything about this?

@Joshpolansky Yes, vaguely. I think we did this because Mapp Motion handles the restore pos in an automated way for you when you map a PV in the drive's configuration?

Was there a reason we didn't use it?

@Joshpolansky
Copy link
Member

Based on testing with MpRobotFlex I'm not sure the drive automatically handles the homing if data is valid. Is that your understanding? I was wondering if it's MpAxisBasic that does the restore pos?

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