feat: X10SA energy range 10-20 keV, compact sample list headers #213

Merged

duan_j merged 3 commits from update-10s-energy-range into main 2026-09-10 17:06:43 +02:00

Conversation 1 Commits 3 Files Changed 3

+20 -16

duan_j commented 2026-09-10 14:43:02 +02:00

Owner

Copy Link

Copy Source

Pick the monochromator energy limits by BEAMLINE env via mx_beamline().
X10SA gets 10-20 keV per the 6124 planner feedback (first commit had 6-30,
corrected in the follow-up), other beamlines keep the 4-20 keV placeholder
until confirmed. Both Set Energy spinboxes and the tooltip follow the
constants.

Also compacts the user sample list: the four count columns (mount, raster,
screening, rotation) use single-glyph headers instead of long labels to
save width, Comment moves ahead of the data-collection params, and
transmission shows the stored value instead of a percent conversion.
Unit test updated to match.

Generated with Claude Code

Pick the monochromator energy limits by BEAMLINE env via mx_beamline(). X10SA gets 10-20 keV per the 6124 planner feedback (first commit had 6-30, corrected in the follow-up), other beamlines keep the 4-20 keV placeholder until confirmed. Both Set Energy spinboxes and the tooltip follow the constants. Also compacts the user sample list: the four count columns (mount, raster, screening, rotation) use single-glyph headers instead of long labels to save width, Comment moves ahead of the data-collection params, and transmission shows the stored value instead of a percent conversion. Unit test updated to match. Generated with [Claude Code](https://claude.com/claude-code)

gitea-actions bot commented 2026-09-10 14:45:04 +02:00

Copy Link

Copy Source

Coverage report (automated)

Total line + branch coverage: 51%
Diff coverage vs main: 100% (minimum 80%)

Full report: coverage artifact on this run.

Coverage report (automated) Total line + branch coverage: 51% Diff coverage vs main: 100% (minimum 80%) Full report: coverage artifact on this run.

duan_j changed title from feat: X10SA energy range 6-30 keV to feat: X10SA energy range 10-20 keV, compact sample list headers 2026-09-10 17:02:02 +02:00

duan_j added 3 commits 2026-09-10 17:04:44 +02:00

feat: X10SA energy range 6-30 keV ... 39631c5206

Pick the mono limits by BEAMLINE env via mx_beamline(): X10SA gets
6-30 keV, other beamlines keep the 4-20 placeholder until confirmed.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

fix: according to 6124 planner feedback, use 10 to 20 keV at PXII b27556e4cf

feat: compact sample list headers, move Comment column ... d5a7074ec1

Count columns use glyphs instead of long headers to save width, Comment
moves ahead of the data-collection params, transmission shown as the
stored value instead of percent.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

duan_j force-pushed update-10s-energy-range from c69522f017 to d5a7074ec1 2026-09-10 17:04:44 +02:00 Compare

duan_j scheduled this pull request to auto merge when all checks succeed 2026-09-10 17:05:18 +02:00

duan_j merged commit d5a7074ec1 into main 2026-09-10 17:06:43 +02:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

Something is not working

duplicate

This issue or pull request already exists

enhancement

New feature

help wanted

Need some help

invalid

Something is wrong

question

More information is needed

wontfix

This won't be fixed

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

buntschu_d (buntschu_d) duan_j (duan_j) glettig_w (glettig_w) gobbo_a (gobbo_a) gotthard_g (gotthard_g) leonarski_f (leonarski_f) mx-gitea olieric (olieric) perl_d (perl_d) takaba_k (takaba_k) wang_m (wang_m)

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: mx/AareDAQ#213