Skip to content

Conversation

@ZryletTC
Copy link
Contributor

Description

  • Primarily changed serverStat's host_from_PV function so that it could be used from other hutches or no hutch. Done by calling on ioctool's new host option
  • Added a host option to ioctool, reorganized some duplicated code, and added error message for unknown command
  • Also changed PATH variable setting in all scripts so that current version of engineering_tools is used over other versions that might be in a user's PATH

Motivation and Context

Resolves LCLSECSD-296

How Has This Been Tested?

Interactively. I believe I've tested all typical cases, but welcome additional testing.

Where Has This Been Documented?

N/A

@ZryletTC
Copy link
Contributor Author

If #95 is completed, support for camera names could be added pretty easily as well

@ZryletTC ZryletTC marked this pull request as draft March 23, 2022 08:38
@ZryletTC
Copy link
Contributor Author

Changing to draft because I want #95 merged first, and there will inevitably be merge conflicts.

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.

1 participant