Skip to content

IMOD processing bypass and file copying#279

Merged
stephen-riggs merged 6 commits into
mainfrom
imod-bypass
Jul 13, 2026
Merged

IMOD processing bypass and file copying#279
stephen-riggs merged 6 commits into
mainfrom
imod-bypass

Conversation

@stephen-riggs

Copy link
Copy Markdown
Contributor

Updates imod dockerfile to include some missing libraries, then runs using -bypass to avoid the java dependency.

Also sets up a mechanism to copy the output files to a higher level. This was requested by b24 as the output folders can be messy and they want an easy way to access the tomograms.

@stephen-riggs stephen-riggs requested a review from tieneupin July 6, 2026 14:12
@codecov

codecov Bot commented Jul 6, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.45455% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 83.39%. Comparing base (b0b700d) to head (bd3375c).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #279      +/-   ##
==========================================
+ Coverage   83.13%   83.39%   +0.25%     
==========================================
  Files          75       75              
  Lines        9291     9580     +289     
  Branches     1264     1352      +88     
==========================================
+ Hits         7724     7989     +265     
- Misses       1075     1093      +18     
- Partials      492      498       +6     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread Dockerfiles/tomo_align Outdated
Comment thread src/cryoemservices/services/denoise.py Outdated
Comment thread src/cryoemservices/services/membrain_seg.py Outdated

@tieneupin tieneupin 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! I don't think running re.match(...) on the path parts in a NumPy array is needed, but the code should work nonetheless.

Comment thread src/cryoemservices/services/tomo_align_aretomo.py Outdated
@stephen-riggs stephen-riggs merged commit cdcc32e into main Jul 13, 2026
12 checks passed
@stephen-riggs stephen-riggs deleted the imod-bypass branch July 13, 2026 10:55
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