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

Add Ability to control Spectrometer Holding Current From EPICS #9

Merged
merged 1 commit into from
Dec 5, 2024
Merged

Add Ability to control Spectrometer Holding Current From EPICS #9

merged 1 commit into from
Dec 5, 2024

Conversation

NSLentz
Copy link
Contributor

@NSLentz NSLentz commented Dec 5, 2024 •
edited
Loading

Description

  • Added boolean to give EPICS a hook to command holding current on and off for the Spectrometer.

Motivation and Context

  • Sometimes they need holding current, sometimes they don't. This makes it not a manual process to switch.

How Has This Been Tested?

  • Builds successfully. Code will be tested when MFX has downtime.

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)

@NSLentz NSLentz requested a review from rcjwoods December 5, 2024 22:28
@rcjwoods rcjwoods merged commit 9434e32 into pcdshub:master Dec 5, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rcjwoods rcjwoods rcjwoods approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@NSLentz @rcjwoods