Skip to content

Metal ID fixes and improvements#324

Merged
pblowey merged 4 commits intomainfrom
improve_metal_id
Oct 16, 2025
Merged

Metal ID fixes and improvements#324
pblowey merged 4 commits intomainfrom
improve_metal_id

Conversation

@pblowey
Copy link
Copy Markdown
Contributor

@pblowey pblowey commented Oct 16, 2025

Adds the following fixes and improvements to the metal_id pipeline:

  • Fixes bug where peaks with negative coordinates were not being parsed and therefore were not uploaded
  • Set a symlink from the trigger function allowing upstream recipes to pass on a descriptive symlink for the metal id processing
  • Handles case where peaks are not found in metal_id
  • Optionally use pipeline_final directory (if specified in the recipe)
  • Specify primary result and log file in the recipe

Handles case where no peaks are found. Creates data class for storing
peak data in and moves unnecessary logic from ispyb function.
Use of pipeline final set by the recipe "pipeline_final" job parameter.

Primary results files to copy to pipeline_final and to upload to ISPyB
set in the recipe too, via the "primary_result_files" job parameter.

Logic set up to optionally use the pipeline_final if it is defined.
Rendered images will be copied to piepline final but will not be
uploaded as an attachment. They get uploaded to the MXMRRunBlob table
instead.
@pblowey pblowey merged commit fbe663f into main Oct 16, 2025
@pblowey pblowey deleted the improve_metal_id branch October 16, 2025 16:14
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.

1 participant