Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Add a dedicated external trigger scan #38

Merged
merged 5 commits into from
Feb 23, 2024
Merged

Conversation

lschall
Copy link
Member

@lschall lschall commented Feb 20, 2024

This PR introduces a dedicated external trigger scan (scan_ext_trigger) for measurements utilizing a TLU. Additionally, the old scan_simple.py is changed to scan_source.py and will not support TLU based measurements further.
This way we want to separate between source measurements and beam test measurements with a TLU setup to avoid confusion in the scan naming and calling.

@lschall lschall added enhancement New feature or request software Related to software maintenance Maintaining and keeping code/functionality up to date labels Feb 20, 2024
@lschall
Copy link
Member Author

lschall commented Feb 20, 2024

This PR addresses issues #30 and #35.

Copy link
Collaborator

@cbespin cbespin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! I just think we should keep the scan_timeout stopping criterium for the external trigger scan, similar to all other projects. I do not have a specific use case i mind but it is easier to deactivate it rather than patch something together if needed.

tjmonopix2/scans/scan_ext_trigger.py Outdated Show resolved Hide resolved
tjmonopix2/scans/scan_ext_trigger.py Show resolved Hide resolved
tjmonopix2/scans/scan_ext_trigger.py Show resolved Hide resolved
Copy link
Collaborator

@cbespin cbespin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good to me. Feel free to merge

@lschall lschall merged commit 9711935 into development Feb 23, 2024
3 checks passed
@lschall lschall deleted the ext_trigger_scan branch February 23, 2024 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request maintenance Maintaining and keeping code/functionality up to date software Related to software
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants