Add script to show how to add a new beamline#203
Merged
padraic-shafer merged 4 commits intoNSLS2:mainfrom Jan 29, 2026
Merged
Conversation
ambarb
previously requested changes
Oct 15, 2025
Contributor
ambarb
left a comment
ambarb
left a comment
There was a problem hiding this comment.
@stuartcampbell and @danielballan thank you so much for this. I have a couple of small things in code, but a general set of process questions:
Should get the user office to agree with some of these things? Or was Cara happy for you to keep following the pattern once you both worked to establish the template?
ambarb
requested review from
HarikaBishai
and removed request for
HarikaBishai
Contributor
|
clear this is a helper to get started (documentation). let's not merge unless we are sure |
padraic-shafer
force-pushed
the
helper-scripts
branch
from
2b3f8bb to
035c188
Compare
padraic-shafer
approved these changes
Jan 29, 2026
Collaborator
padraic-shafer
left a comment
padraic-shafer
left a comment
There was a problem hiding this comment.
Thank you for adding this script. It's very helpful!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add a script that can be used as the basis to create a script to add a new beamline.