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.

Sign up for GitHub

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

Jump to bottom

AxisTunerMixin is subclass of EpicsMotor #318

Closed
prjemian opened this issue May 30, 2020 · 0 comments · Fixed by #319
Closed

AxisTunerMixin is subclass of EpicsMotor #318

prjemian opened this issue May 30, 2020 · 0 comments · Fixed by #319
Assignees
@prjemian
Labels
question
Milestone
1.2.5

Comments

@prjemian
Copy link
Contributor

Is it necessary for AxisTunerMixin() to be a subclass of EpicsMotor()? Prevents its use with ophyd.sim.SynAxis() to create a tunable synthetic axis. This would be useful for testing and tutorials.
@prjemian prjemian added this to the 1.2.4 milestone May 30, 2020
@prjemian prjemian self-assigned this May 30, 2020
prjemian added a commit that referenced this issue Jun 1, 2020
@prjemian
fixes #318
ff4ca4a
@prjemian prjemian mentioned this issue Jun 1, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@prjemian prjemian

Labels
question
Projects
None yet
Milestone
1.2.5
Development

Successfully merging a pull request may close this issue.

AxisTunerMixin should use more generic parent BCDA-APS/apstools
1 participant
@prjemian