Skip to content

Added Dockerfile to build image that will run PySIMRecon - #293

Merged
tieneupin merged 3 commits into
mainfrom
add-pysimrecon-dockerfile
Aug 17, 2026
Merged

Added Dockerfile to build image that will run PySIMRecon#293
tieneupin merged 3 commits into
mainfrom
add-pysimrecon-dockerfile

Conversation

@tieneupin

@tieneupin tieneupin commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Adds a new Dockerfile to build a container that can run PySIMRecon as a cryoem-services service. The container, when run using podman run ..., has been confirmed to work on existing SIM data.

The next PR will involve configuring the PySIMRecon service to run PySIMRecon in a subprocess.

@codecov

codecov Bot commented Aug 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.32%. Comparing base (73061a6) to head (8168341).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #293      +/-   ##
==========================================
+ Coverage   83.06%   83.32%   +0.25%     
==========================================
  Files          76       76              
  Lines        9584     9931     +347     
  Branches     1289     1385      +96     
==========================================
+ Hits         7961     8275     +314     
- Misses       1120     1143      +23     
- Partials      503      513      +10     
🚀 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 August 11, 2026 10:55
@tieneupin tieneupin self-assigned this Aug 11, 2026
@tieneupin tieneupin added the enhancement New feature or request label Aug 11, 2026
@tieneupin
tieneupin force-pushed the add-pysimrecon-dockerfile branch from eefac82 to eeb75af Compare August 12, 2026 02:42

@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.

All looks fine. I'm not very familiar with what you're doing with mounting into /mnt but as long as the dockerfile works I'm sure it's fine

@tieneupin

tieneupin commented Aug 17, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for reviewing!

I'm not very familiar with what you're doing with mounting

This is an alternative I've found to using COPY ... when moving setup files from the build directory to the image. You can instead grant the image access to the build directory and mount it to a specific path. This would in practice help with reducing the final image size, since you've removed one layer from the Dockerfile.

@tieneupin
tieneupin merged commit 1098698 into main Aug 17, 2026
10 checks passed
@tieneupin
tieneupin deleted the add-pysimrecon-dockerfile branch August 17, 2026 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants