Skip to content

comprehensive update#1

Merged
brechmos merged 1 commit intobrechmos:mainfrom
walkerbdev:feature/v0.2-improvements
Apr 21, 2026
Merged

comprehensive update#1
brechmos merged 1 commit intobrechmos:mainfrom
walkerbdev:feature/v0.2-improvements

Conversation

@walkerbdev
Copy link
Copy Markdown
Contributor

  • Restructured flat scripts into a datastore_statistics/ package
  • Added CLI (cli.py), file readers (readers.py), image quality metrics (stats.py), HTML reports (report.py), and utilities (utils.py)
  • Added pyproject.toml, pre-commit hooks, and updated .gitignore
  • Added comprehensive test suite with shared fixtures
  • Expanded README with installation, usage, and metrics docs
  • Multimetric outlier detection (compute outlier detection for multiple metrics vs only single metric)
    New metrics:
  • Spatial: n_slices, matrix_x/y, voxel_x/y/z, fov_x/y/z
  • Center of mass: com_x/y/z (in mm when voxel spacing is available)
  • Image quality: efc (Entropy Focus Criterion) and fber (Foreground-Background Energy Ratio)
  • Masking: foreground_pct, auto_threshold (Otsu)

@walkerbdev walkerbdev force-pushed the feature/v0.2-improvements branch 2 times, most recently from e1ffdce to e7ba0a2 Compare January 29, 2026 15:32
Copy link
Copy Markdown

@hamshkhawar hamshkhawar left a comment

Choose a reason for hiding this comment

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

@walkerbdev It looks good. Few suggestion mainly adding type annotations for input prams and return values

Comment thread datastore_statistics/cli.py Outdated
Comment thread tests/__init__.py
Comment thread datastore_statistics/stats.py Outdated
Comment thread datastore_statistics/utils.py Outdated
Comment thread datastore_statistics/utils.py Outdated
@walkerbdev walkerbdev force-pushed the feature/v0.2-improvements branch 10 times, most recently from 5701da9 to 900954e Compare February 5, 2026 19:03
@walkerbdev walkerbdev force-pushed the feature/v0.2-improvements branch from 900954e to 73e0bd1 Compare February 9, 2026 16:03
@walkerbdev walkerbdev force-pushed the feature/v0.2-improvements branch from 73e0bd1 to f2928c3 Compare February 27, 2026 17:26
@brechmos brechmos merged commit 25af213 into brechmos:main Apr 21, 2026
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.

3 participants