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

ENH: tmo spectrometer uses TMO:SPEC as happi prefix #132

Closed
wants to merge 1 commit into from
Closed

ENH: tmo spectrometer uses TMO:SPEC as happi prefix #132

wants to merge 1 commit into from

Conversation

nrwslac
Copy link
Contributor

@nrwslac nrwslac commented Sep 13, 2024 •
edited
Loading

Description

change flow meter PV name to match TMO-Motion Spectrometer style:
- {attribute 'pytmc' :='pv: SP1K4'}
+ {attribute 'pytmc' :='pv: TMO:SPEC'}

Motivation and Context

How Has This Been Tested?

Where Has This Been Documented?

Screenshots (if appropriate):

Pre-merge checklist

  • Code works interactively
  • Code contains descriptive comments
  • Test suite passes locally
  • Libraries are set to fixed versions and not Always Newest
  • Code committed with pre-commit (alternatively pre-commit run --all-files)

@nrwslac nrwslac marked this pull request as ready for review September 13, 2024 22:50
@nrwslac nrwslac mentioned this pull request Sep 13, 2024
Merged
7 tasks
@nrwslac nrwslac closed this Sep 17, 2024
@nrwslac
Copy link
Contributor Author

nrwslac commented Sep 17, 2024

Closed, changes made in #133

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tongju12 tongju12 Awaiting requested review from tongju12

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nrwslac