Skip to content

Review python dependencies#488

Merged
Gab-D-G merged 2 commits intomasterfrom
review_python_dependencies
Mar 25, 2026
Merged

Review python dependencies#488
Gab-D-G merged 2 commits intomasterfrom
review_python_dependencies

Conversation

@Gab-D-G
Copy link
Copy Markdown
Collaborator

@Gab-D-G Gab-D-G commented Mar 25, 2026

Summary by CodeRabbit

  • Chores
    • Pinned package dependencies to specific commit hashes across build configurations for consistent and reproducible installations
    • Updated pip minimum version requirement to >=23.0

@Gab-D-G Gab-D-G merged commit 690574a into master Mar 25, 2026
2 checks passed
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 25, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a6505064-8c11-48d6-b6d4-b4371931bac4

📥 Commits

Reviewing files that changed from the base of the PR and between 1998a33 and 26ec6ed.

📒 Files selected for processing (3)
  • Dockerfile
  • rabies_environment.yml
  • setup.py

📝 Walkthrough

Walkthrough

This pull request pins dependency versions for reproducibility and stability. The simpleitk_timeseries_motion_correction Git dependency is pinned to a specific commit hash across Dockerfile and setup.py, while pip is versioned to >= 23.0 in environment and setup configurations.

Changes

Cohort / File(s) Summary
Git dependency pinning
Dockerfile, setup.py
Pinned simpleitk_timeseries_motion_correction from moving ref rabies_conform to fixed commit 8cc2b031c22df5cf8d01f75ec186570dd3ffb3e9 for deterministic builds.
Pip version constraint
rabies_environment.yml, setup.py
Added minimum pip version constraint pip>=23.0 to Conda environment and required installation dependencies.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • gdevenyi

Poem

🐰 Ah, pinned commits dance with glee,
No more moving refs, just certainty!
Pip version locked, the builds align,
Reproducibility—now that's divine!
Hop, hop, hooray for stable deps,
We've conquered the chaos in single steps! 🥕✨

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch review_python_dependencies

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant