Skip to content

Conversation

@dthanasekaran
Copy link
Contributor

Description

one line change for UED detname for new epix

Motivation and Context

new epix change in ued

How Has This Been Tested?

by pointing engineering tool to my dev directory

Where Has This Been Documented?

@dthanasekaran dthanasekaran requested a review from a team as a code owner October 24, 2025 19:29
@tangkong
Copy link
Contributor

The pre-commit failures are in the file that was changed, but are unrelated to the proposed change.

We should continue shellchecking this repo though, as an aside.

@patoppermann
Copy link
Contributor

Do we never go back to the "normal" epix?

Copy link
Collaborator

@silkenelson silkenelson left a comment

Choose a reason for hiding this comment

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

I am still hoping that we will be able to move all the ideally needed smarts into a DAQ script, so let's hardcode this new detector and make sure UED knows that the old one won't work with takepeds anymore now.

@dthanasekaran dthanasekaran merged commit 11b4886 into pcdshub:master Oct 24, 2025
1 of 2 checks passed
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.

4 participants