Skip to content

Conversation

@razvanvasile
Copy link
Contributor

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.016% when pulling 009fc8c on razvanvasile:enabled_devices into 02a8e9c on willrogers:master.

2 similar comments
@coveralls
Copy link

coveralls commented May 24, 2017

Coverage Status

Coverage remained the same at 93.016% when pulling 009fc8c on razvanvasile:enabled_devices into 02a8e9c on willrogers:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.016% when pulling 009fc8c on razvanvasile:enabled_devices into 02a8e9c on willrogers:master.

pytac/device.py Outdated
raise PvException("Readback or setpoint pvs need to be given")

def is_enabled(self):
"""Check whether an element is enabled or disabled.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to update the comments to refer to devices not elements...

@coveralls
Copy link

coveralls commented May 24, 2017

Coverage Status

Coverage remained the same at 93.016% when pulling 4af26bc on razvanvasile:enabled_devices into 02a8e9c on willrogers:master.

@coveralls
Copy link

coveralls commented May 24, 2017

Coverage Status

Coverage decreased (-0.3%) to 92.698% when pulling fa94c95 on razvanvasile:enabled_devices into 02a8e9c on willrogers:master.

@willrogers willrogers merged commit 451efd7 into DiamondLightSource:master May 24, 2017
@razvanvasile razvanvasile deleted the enabled_devices branch May 25, 2017 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants