bec/xtreme_bec
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Feature/mirror pseudo #25

Merged
wakonig_k merged 4 commits from feature/mirror_pseudo into main 2026-07-14 16:50:23 +02:00
Conversation 4 Commits 4 Files Changed 3
+115 -146
wakonig_k commented 2026-07-13 17:46:41 +02:00
Owner
Copy Link
Copy Source

@piamonteze I dropped the added PGM_ID_coupling signal as it was anyway part of the pgm device. We can add it again easily if you want though.

@piamonteze I dropped the added PGM_ID_coupling signal as it was anyway part of the pgm device. We can add it again easily if you want though.
wakonig_k added 1 commit 2026-07-13 17:46:41 +02:00
chore(otf): move pre scan after setup 906d7dc5cc
wakonig_k requested review from piamonteze 2026-07-13 17:47:05 +02:00
wakonig_k added 1 commit 2026-07-13 18:03:59 +02:00
feat: adding bespoke pseudo motor to CMU and FMU
CI for xtreme_bec / test (pull_request) Successful in 34s
Details
CI for xtreme_bec / test (push) Successful in 34s
Details
5fddf24901
wakonig_k force-pushed feature/mirror_pseudo from 7835af861f to 5fddf24901 2026-07-13 18:03:59 +02:00 Compare
piamonteze commented 2026-07-14 09:10:54 +02:00
Member
Copy Link
Copy Source

Hi @wakonig_k. I would like to keep the PGM+ID channel in. I use it now in the harmonic macro. I prefer to keep the PGM synchronization channel exclusively being used by the on-the-fly scan. Perhaps it wouldn't make a difference, it's more a habit thing.

Hi @wakonig_k. I would like to keep the PGM+ID channel in. I use it now in the harmonic macro. I prefer to keep the PGM synchronization channel exclusively being used by the on-the-fly scan. Perhaps it wouldn't make a difference, it's more a habit thing.
👍 1
wakonig_k added 1 commit 2026-07-14 09:50:26 +02:00
feat: add PGM ID coupling device configuration
CI for xtreme_bec / test (pull_request) Successful in 36s
Details
CI for xtreme_bec / test (push) Successful in 1m2s
Details
1acc61ddb7
wakonig_k commented 2026-07-14 09:52:52 +02:00
Author
Owner
Copy Link
Copy Source

I've added it again. In general, we should think about the naming conventions for X-Treme. Right now, there is a bit of a mixture between lower and upper case devices, e.g. CMU_z and PGM_ID_coupling but mirror_theta and grating_beta.

I've added it again. In general, we should think about the naming conventions for X-Treme. Right now, there is a bit of a mixture between lower and upper case devices, e.g. CMU_z and PGM_ID_coupling but mirror_theta and grating_beta.
piamonteze commented 2026-07-14 16:45:10 +02:00
Member
Copy Link
Copy Source

Hi @wakonig_k . I have "loaded current configuration" and I did not see the pgm+id. About the naming convention, it's true. It looks a bit messy. All low case as you were doing is fine for me. Perhaps this would be a good exercise for me to do with git but probably better to do with you.

Hi @wakonig_k . I have "loaded current configuration" and I did not see the pgm+id. About the naming convention, it's true. It looks a bit messy. All low case as you were doing is fine for me. Perhaps this would be a good exercise for me to do with git but probably better to do with you.
👍 1
wakonig_k commented 2026-07-14 16:46:24 +02:00
Author
Owner
Copy Link
Copy Source

Yeah, this is only on git now. Let's merge it and I quickly update your session on x07ma

Yeah, this is only on git now. Let's merge it and I quickly update your session on x07ma
wakonig_k added 1 commit 2026-07-14 16:48:33 +02:00
fix: normalize device names to lowercase in optics configuration
CI for xtreme_bec / test (pull_request) Successful in 1m12s
Details
CI for xtreme_bec / test (push) Successful in 33s
Details
509ca6a195
wakonig_k merged commit 509ca6a195 into main 2026-07-14 16:50:23 +02:00
wakonig_k deleted branch feature/mirror_pseudo 2026-07-14 16:50:30 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
piamonteze
Labels
Clear labels
issue::not confirmed
Used if an issue has not yet been confirmed by a developer
 issue::not reproducible
Used if an issue is not reproducible
 issue::reproducible
Used if a developer was able to reproduce the issue
 priority::critical
Should be done as soon as possible
 priority::needed
An important update to the code base
 priority::nice to have
Not really needed but a good improvement
 scope::backend
An issue regarding the backend services
 scope::frontend
An issue regarding the user-facing services
 stage::in development
This issue is currently being worked on
 stage::on hold
Something is blocking the development
 stage::pre_release
For issues that have been merged into a pre-release branch and are waiting for their release.
stage::ready for review
For issues that have been resolved and are ready for review
 stage::selected for development
This issue has been selected for development but has not been scheduled yet
 stage::weekly backlog
This issue has been selected for development for the next development cycle
 type::bug
It's a bug!
 type::discussion
A discussion is wanted/needed
 type::doc
Improvements to the documentation
 type::feature
A new feature
 type::improvement
Neither a feature nor a bug but an improvement to the code base
 type::test
Missing tests or improvements to the test suite
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
acerbo_a (acerbo_a) appel_c (appel_c) bec-gitea celcer_t (celcer_t) debenjak_l (debenjak_l) divall_e (divall_e) haghta_s (haghta_s) lombosi_c (lombosi_c) perl_d (perl_d) piamonteze (piamonteze) raabe (raabe) usov_i (usov_i) wakonig_k (wakonig_k) wang_x1 (wang_x1) wyzula_j (wyzula_j) zamofing_t (zamofing_t)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: bec/xtreme_bec#25