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

spec2ophyd: motors do not appear in *wa* command #202

Closed
prjemian opened this issue Jul 27, 2019 · 2 comments · Fixed by #208
Closed

spec2ophyd: motors do not appear in *wa* command #202

prjemian opened this issue Jul 27, 2019 · 2 comments · Fixed by #208
Assignees
@prjemian
Labels
bug
Milestone
1.1.10

Comments

@prjemian
Copy link
Contributor

prjemian commented Jul 27, 2019 •
edited
Loading

Any instances of EpicsMotor created by the spec2ophyd tool do not appear in the wa (report all known motors). This is a SPEC compatibility command from ophyd. Since it is expected any such motors should appear in the wa command, this is a bug.
@prjemian prjemian added the bug label Jul 27, 2019
@prjemian prjemian added this to the milestone-2019-09 milestone Jul 27, 2019
@prjemian prjemian self-assigned this Jul 27, 2019
@prjemian
Copy link
Contributor Author

prjemian commented Jul 27, 2019 •
edited
Loading

To enable their report in wa, add a labels attribute, such as:

m4 = EpicsMotor('vm7:m4', name='m4', labels=("motor", "fourc"))
m5 = EpicsMotor('vm7:m5', name='m5', labels=("motor",))

prjemian added a commit that referenced this issue Jul 29, 2019
@prjemian
fixes #202
257c315
@prjemian
Copy link
Contributor Author

Same for ScalerCH and EpicsSignal

@prjemian prjemian mentioned this issue Jul 29, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@prjemian prjemian

Labels
bug
Projects
None yet
Milestone
1.1.10
Development

Successfully merging a pull request may close this issue.

add labels attribute to enable wa and ct magic commands BCDA-APS/apstools
1 participant
@prjemian