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

LODCM - additional functions #763

Open
5 of 8 tasks
cristinasewell opened this issue Feb 18, 2021 · 3 comments
Open
5 of 8 tasks

LODCM - additional functions #763

cristinasewell opened this issue Feb 18, 2021 · 3 comments

Comments

@cristinasewell
Copy link
Contributor

cristinasewell commented Feb 18, 2021

Expected Behavior

The following functions might be something scientist might want for the LODCM:

We might need some more feedback from scientists about other things we should include in the LODCM object after #715 is merged in.

Work on fixing/changing the following suggestions - might need to be split into multiple PRs.

Current Behavior

We don't have those functions.

Note

In the old code we use the get_par('retry_deadband') - which I am not sure what it does yet exactly...
if abs(dx - x) > self.x2.get_par("retry_deadband"):

@ZLLentz
Copy link
Member

ZLLentz commented Feb 18, 2021

get_par('retry_deadband')

Was basically a caget of an EPICS PV, $PREFIX.RDBD

According to the motor record manual, this is the "retry deadband" field that says "retry moves if they did not get at least this close to the goal"

@cristinasewell
Copy link
Contributor Author

cristinasewell commented Feb 23, 2021

Work on these guys here:

#715 (review)
#715 (comment)
#715 (comment)
maybe others

@cristinasewell
Copy link
Contributor Author

cristinasewell commented Feb 24, 2021

@dlzhu when you get a chance, could you please let us know what other functions you use with LODCM other than the ones mentioned here? (tweakX, tweakParallel, setE, waitE)

I think all these functions I got from here:
https://github.com/pcdshub/old-hutch-configs/blob/18c7769d66a647ab558ad7ebc34d019fc98fbf48/xpp/blinst/lom.py#L1093

Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants