Skip to content

Partial udc#73

Merged
fhernandezvivanco merged 17 commits into
mainfrom
partial_udc
Dec 1, 2025
Merged

Partial udc#73
fhernandezvivanco merged 17 commits into
mainfrom
partial_udc

Conversation

@fhernandezvivanco

@fhernandezvivanco fhernandezvivanco commented Dec 1, 2025

Copy link
Copy Markdown
Collaborator
  • Creates a dialog box schema folder where all json schemas are saved. These schemas are used by partial UDC and inidividual scans
  • Adds the option to trigger partial UDC. The user can select to run:
  1. Optical and X-ray Centering
  2. Optical and X-ray Centering + Screening
  3. Optical and X-ray Centering + Full Dataset
  4. Optical and X-ray Centering + Screening + Full Dataset

),
photon_energy=photon_energy,
transmission=full_dataset_model.transmission / 100,
beam_size=(80, 80),

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Is there a PV or redis key available yet to get the beam size?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Not yet as far as I know. I'll double check



class PartialUDCFlow(AbstractPrefectWorkflow):
"""Partial UDC Prefect Flow. Allows running:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Can the option to run just Optical centering be available too?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

yep, that should be easy to add :)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

actually that will require changing the json schemas of the dialog boxes, so we can leave that for the next PR so that this can be tested in stage tomorrow?

@katesmith280 katesmith280 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good! I agree let's put the optical centering in the next PR so we can test all plans requiring X-rays at the beamline before shutdown (optical centering can be tested without beam)

@fhernandezvivanco fhernandezvivanco merged commit aa37146 into main Dec 1, 2025
3 of 11 checks passed
@fhernandezvivanco fhernandezvivanco deleted the partial_udc branch December 1, 2025 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants