Skip to content

Count preprocessed movies - #921

Merged
tieneupin merged 13 commits into
mainfrom
count-preprocessed-movies
Oct 13, 2026
Merged

tieneupin merged 13 commits into
mainfrom
count-preprocessed-movies

Conversation

@tieneupin

@tieneupin tieneupin commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Updating the last_active time as part of processing, introduced in PR #913, was a step in the right direction. However, it is reliant on processing occurring as planned; a processing failure late at night or over the weekend could lead to the timestamp failing to update, making it seem as if the session has become inactive.

A more robust check for the SPA and tomo workflows for whether processing is complete would involve counting how many of the movies associated with the current session have yet to undergo preprocessing (Movie.preprocessed == False). The number of complete vs incomplete movies can then be passed along to the Murfey frontend to warn the user whether to proceed with a session cleanup or not.

This PR adds a new API endpoint to the murfey.server.api.session_info module that returns the number of complete and incomplete movies for the specified session as a dictionary. It also adds logic when registering motion_corrected messages to also update the Movie rows associated with the received tilt series and mark it as having been preprocessed. Also adds a test for the new API endpoint.

@codecov

codecov Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 71.42857% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.45%. Comparing base (3fed4b7) to head (42377db).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #921      +/-   ##
==========================================
+ Coverage   55.43%   55.45%   +0.01%     
==========================================
  Files         103      103              
  Lines       11636    11650      +14     
  Branches     1546     1549       +3     
==========================================
+ Hits         6450     6460      +10     
- Misses       4837     4841       +4     
  Partials      349      349              
🚀 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 12, 2026 21:52
@tieneupin tieneupin self-assigned this Oct 12, 2026
@tieneupin tieneupin added enhancement New feature or request server Relates to the server component tomography Part of CryoET workflows labels Oct 12, 2026

@stephen-riggs stephen-riggs left a comment

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.

Looks good to me

@tieneupin
tieneupin merged commit 33bfa76 into main Oct 13, 2026
9 checks passed
@tieneupin
tieneupin deleted the count-preprocessed-movies branch October 13, 2026 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request server Relates to the server component tomography Part of CryoET workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants