Skip to content

Trigger processing on cryo-SIM atlas files - #925

Open
tieneupin wants to merge 5 commits into
mainfrom
process-sim-atlas
Open

tieneupin wants to merge 5 commits into
mainfrom
process-sim-atlas

Conversation

@tieneupin

@tieneupin tieneupin commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

This PR adds logic to the Analyser and SIMContext to detect the atlas files and request for them to be processed. A stub FastAPI endpoint has been added to the backend that will log when processing parameters are received. This will be extended in subsequent PRs.

In the cryo-SIM workflow, the atlas stitching service requires three files:

  • A mosaic.mrc file, which contains the actual image frames
  • A mosaic.txt file, which points to the MRC file and contains the stage position information for each frame
  • An optional markers.txt file, which contains annotation information to apply to the stitched image

As long as the two mosaic files are accounted for, a processing request will be sent to the backend. If a markers file is created later on, a new processing request will be sent which will regenerate the image with the annotations.

@codecov

codecov Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.33333% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.83%. Comparing base (da1390a) to head (8123597).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #925      +/-   ##
==========================================
+ Coverage   55.74%   55.83%   +0.08%     
==========================================
  Files         103      103              
  Lines       11643    11671      +28     
  Branches     1553     1560       +7     
==========================================
+ Hits         6490     6516      +26     
- Misses       4802     4803       +1     
- Partials      351      352       +1     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tieneupin
tieneupin marked this pull request as ready for review October 22, 2026 15:40
@tieneupin tieneupin self-assigned this Oct 22, 2026
@tieneupin tieneupin added client Relates to the client component cryo-sim Part of the cryo-SIM pipeline extension enhancement New feature or request labels Oct 22, 2026
or (
# CryoSIM consist of a mosaic MRC file and an accompanying text file
# Both should have the word "mosaic" in them
"mosaic" in file_path.stem and file_path.suffix in (".mrc", ".txt")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Might need to work out what we have across different instruments here, as the .mrc extension might conflict with other things

@stephen-riggs
stephen-riggs self-requested a review October 23, 2026 16:08

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

client Relates to the client component cryo-sim Part of the cryo-SIM pipeline extension enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants