Skip to content

Conversation

@tieneupin
Copy link
Contributor

@tieneupin tieneupin commented Feb 6, 2026

The recently released version of ispyb-api includes new columns in the Atlas and GridSquare tables that can be used to keep track of the data collection mode used for, and the colour channels present in, a particular dataset.

This PR adds optional logic to the Atlas and GridSquare insertion/update functions to register the colour channel information and collection mode associated with the CLEM workflow. This involves adding new columns to the CLEMImageSeries Murfey database table. Additionally, it fixes an error with how the size of the ROI on the atlas image is calculated.

@codecov
Copy link

codecov bot commented Feb 6, 2026

Codecov Report

❌ Patch coverage is 87.93103% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 46.91%. Comparing base (8fb34a2) to head (d7320d4).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #739      +/-   ##
==========================================
+ Coverage   46.65%   46.91%   +0.25%     
==========================================
  Files          91       91              
  Lines        9607     9665      +58     
  Branches     1255     1270      +15     
==========================================
+ Hits         4482     4534      +52     
- Misses       4912     4914       +2     
- Partials      213      217       +4     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tieneupin tieneupin force-pushed the register-clem-colours branch from f109365 to 095520d Compare February 6, 2026 07:27
@tieneupin tieneupin self-assigned this Feb 6, 2026
@tieneupin tieneupin added server Relates to the server component cryo-clem Part of the cryo-CLEM pipeline extension labels Feb 6, 2026
Copy link
Contributor

@d-j-hatton d-j-hatton left a comment

Choose a reason for hiding this comment

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

Looks good. Looking cool in PATo so good to progress this

tieneupin and others added 20 commits February 10, 2026 12:42
… the Atlas and GridSquare tables in ISPyB for the CLEM workflow
…of just activating the ones it receives; fixed colour flag and collection mode insertion logic for the Atlas
…rival of message, then translate it across to Atlas and GridSquare tables in ISPyB
@tieneupin tieneupin force-pushed the register-clem-colours branch from d121b9e to d7320d4 Compare February 10, 2026 12:44
@tieneupin tieneupin requested a review from d-j-hatton February 10, 2026 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cryo-clem Part of the cryo-CLEM pipeline extension server Relates to the server component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants