Conversation
…v09 (#81) Issue #81 asked to drop the per-server subdirectory layer inside each location. PR #107 (04c4a76, Sep 2019) flattened stanford/smurf-srv03 and smurf-srv04, and 9037791 (Nov 2019) flattened pton/smurf-srv08, but nist/xray/smurf-srv09 was missed. Move its lone cfg file up to nist/xray/ so the layout matches every other location (b33, kx, lab1, nist/cmb, pton, rflab, slac, stanford, ucsd, template). The filename is unchanged so existing user scripts that reference it (e.g. scratch/shawn/scripts/init_nist_xray.py) keep working.
swh76
approved these changes
May 26, 2026
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
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.
Summary
smurf-srv*/subdirectory undercfg_files/.cfg_files/nist/xray/smurf-srv09/experiment_nistxray_srv09_dspv3_cc02-05_lbOnlyBay0.cfgup tocfg_files/nist/xray/, matching the flat layout used by every other location.scratch/shawn/scripts/init_nist_xray.py) keep working.Background
Issue #81 (Aug 2019) asked to drop the per-server subdirectory layer inside each location. The bulk of that work was done years ago:
stanford/smurf-srv03/andstanford/smurf-srv04/.pton/smurf-srv08/.nist/xray/smurf-srv09/was missed. This PR finishes the cleanup. After merge,find cfg_files -type d -name 'smurf-srv*'returns nothing.The naming-guideline portion of the original issue was deliberately scoped to confluence by the reporter ("set some guideline ... and post that to the confluence"); no in-repo action is needed there.
Test plan
find cfg_files -type d -name 'smurf-srv*'returns nothinggit log --followon the moved file shows preserved historyflake8 --count python/returns 0 (no Python touched, but verified)