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

add findpv(PVNAME) utility function #497

Merged
merged 18 commits into from
Apr 1, 2021
Merged

add findpv(PVNAME) utility function #497

merged 18 commits into from
Apr 1, 2021

Conversation

prjemian
Copy link
Contributor

@prjemian prjemian commented Mar 2, 2021

Add utils.findpv() to fix #496

This needs to become part of `utils` but should go into a (new) `_utils` subdirectory.  Also needs documentation and tests.
@prjemian prjemian added this to the 1.4.2 milestone Mar 2, 2021
@prjemian prjemian self-assigned this Mar 2, 2021
@lgtm-com
Copy link

lgtm-com bot commented Mar 2, 2021

This pull request introduces 1 alert when merging 09b13a7 into 0fa887e - view on LGTM.com

new alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Mar 14, 2021

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

new alerts:

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

@prjemian prjemian marked this pull request as ready for review March 31, 2021 23:43
@prjemian
Copy link
Contributor Author

prjemian commented Apr 1, 2021

@gfabbris : Ready for review

Copy link
Collaborator

@gfabbris gfabbris left a comment

Choose a reason for hiding this comment

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

Looks great!!

apstools/utils.py Outdated Show resolved Hide resolved
@prjemian prjemian merged commit 6c00530 into main Apr 1, 2021
@prjemian prjemian deleted the 496_find_EPICS_PV branch April 1, 2021 22:34
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 ophyd signal uses this EPICS PV?
2 participants