Skip to content

Beckhoff: Implement AUTO_ONOFF #6839

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

Open
6 tasks
DominicOram opened this issue Oct 20, 2021 · 7 comments
Open
6 tasks

Beckhoff: Implement AUTO_ONOFF #6839

DominicOram opened this issue Oct 20, 2021 · 7 comments
Labels

Comments

@DominicOram
Copy link
Contributor

DominicOram commented Oct 20, 2021

As a user of a Beckhoff controller I would like the option to set my axes to automatically energise/de-energise between moves or to set them to always be on. This functionality is already there in the GALIL as AUTO_ONOFF. The Beckhoff variable for setting whether the motor is on or not is BENABLE.

Acceptance Criteria

  • The user can set OFF_CMD to turn the Beckhoff axis off
  • The user can set ON_CMD to turn the Beckhoff axis on
  • The user can set AUTOONOFF_CMD to 1 and the Beckhoff axis will turn on before a move and off again after
  • The user can set AUTOONOFF_CMD to 0 and the on/off status of the Beckhoff axis will be unchanged by a move
  • AUTOONOFF_STATUS and ON_STATUS also exist to show current device status
  • These commands, and readbacks are available on the OPI, like with the galil
@FreddieAkeroyd
Copy link
Member

Just to clarify, BENABLE is the on/off variable to set, but the "auto" feature needs to be done in our driver? Also is BENABLE read/write i.e. if the motor is turned off via anther route will it update to reflect the current status?

@FreddieAkeroyd
Copy link
Member

@DominicOram
Copy link
Contributor Author

Yes, currently we need to do the auto bit ourselves, I'll talk to Simon about whether it makes sense to put it in the PLC. Yes, it's read/write.

@rerpha
Copy link
Contributor

rerpha commented Nov 5, 2021

So far what i've got is:

ISISComputingGroup/EPICS-tcIoc#10
ISISComputingGroup/EPICS-ioc#651

note: after testing if this works OK remove the print

@DominicOram
Copy link
Contributor Author

This looks like it would work if it wasn't for #6874

@rerpha
Copy link
Contributor

rerpha commented Jan 4, 2022

#6874 has solved DMOV and MOVN i believe so this may work, i'll test and put in review if it does

@rerpha
Copy link
Contributor

rerpha commented Jan 7, 2022

ok, still needs some work, cant seem to get it to disable the axis now for some reason and we need to propagate it through to the GUI, so im going to shelve for now in favour of other work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: To do
Development

No branches or pull requests

4 participants