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

Show which Devices and Signals are defined #195

Merged
merged 10 commits into from
Jul 25, 2019
Merged

Conversation

prjemian
Copy link
Contributor

fixes #194

@prjemian prjemian added this to the milestone-2019-09 milestone Jul 25, 2019
@prjemian prjemian self-assigned this Jul 25, 2019
@prjemian
Copy link
Contributor Author

Needs a unit test

@prjemian
Copy link
Contributor Author

example output:

In [10]: show_ophyd_symbols(True)                                                                                                                                                                      
======== ================================ =============
name     class                            EPICS PV     
======== ================================ =============
adsimdet MySingleTriggerSimDetector       vm7SIM1:     
m1       EpicsMotor                       vm7:m1       
m2       EpicsMotor                       vm7:m2       
m3       EpicsMotor                       vm7:m3       
m4       EpicsMotor                       vm7:m4       
m5       EpicsMotor                       vm7:m5       
m6       EpicsMotor                       vm7:m6       
m7       EpicsMotor                       vm7:m7       
m8       EpicsMotor                       vm7:m8       
noisy    EpicsSignalRO                    vm7:userCalc1
scaler   ScalerCH                         vm7:scaler1  
shutter  SimulatedApsPssShutterWithStatus              
======== ================================ =============

Out[10]: <pyRestTable.rest_table.Table at 0x7fbbe6a7fb38>

@prjemian
Copy link
Contributor Author

Set default show_pv to True, probably more typical interest level.

@lgtm-com
Copy link

lgtm-com bot commented Jul 25, 2019

This pull request introduces 2 alerts when merging 2de3784 into ef3e900 - view on LGTM.com

new alerts:

  • 2 for Missing call to __init__ during object initialization

prjemian added 3 commits July 25, 2019 12:24
* show PV by default
* printing is optional now
@lgtm-com
Copy link

lgtm-com bot commented Jul 25, 2019

This pull request introduces 1 alert when merging 0b5226d into c5292eb - view on LGTM.com

new alerts:

  • 1 for Module is imported with 'import' and 'import from'

@prjemian prjemian modified the milestones: milestone-2019-09, 1.1.8 Jul 25, 2019
@lgtm-com
Copy link

lgtm-com bot commented Jul 25, 2019

This pull request fixes 2 alerts when merging 9211c6d into c5292eb - view on LGTM.com

fixed alerts:

  • 2 for Missing call to __init__ during object initialization

@prjemian prjemian merged commit f1e15e8 into master Jul 25, 2019
@prjemian prjemian deleted the 194-show-symbols branch July 25, 2019 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

What Devices and Signals are defined?
1 participant