Skip to content

fix: inference and analysis workflow bugs#60

Merged
jtschwar merged 2 commits into
mainfrom
fixes
Jun 23, 2026
Merged

fix: inference and analysis workflow bugs#60
jtschwar merged 2 commits into
mainfrom
fixes

Conversation

@jtschwar

@jtschwar jtschwar commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fixes several bugs in the inference and analysis workflows surfaced during user testing.

  • inference_core.py: Fixed incorrect dimension check - the condition image.ndim == 2 was guarding a 3D slice [:,:,0], causing a crash on 2D images. Now correctly checks if the output is 3D before slicing.
  • run_tomogram_segment.py: Fixed parameter name mismatch - run_ids was passed where the function expected run_id.
  • results.py: Fixed a crash in save_slab_seg when a run string contains more than one - (e.g. TS_001-0-1). The split is now handled safely, joining any extra segments back into the session ID.
  • run_analysis.py: Added -o/--output option to the coordinates, statistics, and slurm commands to allow users to specify the output CSV path (previously hardcoded as {organelle_name}_statistics.csv). Also added no_args_is_help=True to the coordinates and statistics commands.

Acknowledgements

Thanks to @KatrinaABlack and @jowagner91 for identifying these bugs and providing recommendations for the fixes.

@jtschwar
jtschwar merged commit f477380 into main Jun 23, 2026
8 checks passed
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