Skip to content

Conversation

@jan-janssen
Copy link
Member

@jan-janssen jan-janssen commented Oct 15, 2025

Summary by CodeRabbit

  • Chores
    • Upgraded h5py to 3.15.0 across CI, environment, binder, and optional-extras configurations to standardize versions.
    • Aligns Linux, Windows, MPI, and interactive setups to reduce drift and improve reproducibility.
    • No functional or API changes expected; may yield minor stability or performance improvements in data I/O.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 15, 2025

Walkthrough

Dependency version bump of h5py across CI environment files, Binder environment, and pyproject optional dependencies. No functional, control-flow, or code logic changes.

Changes

Cohort / File(s) Summary of changes
CI environments
.ci_support/environment-docs.yml, .ci_support/environment-mpich.yml, .ci_support/environment-openmpi.yml, .ci_support/environment-win.yml
Bump h5py from 3.14.0 to 3.15.0.
Binder environment
binder/environment.yml
Bump h5py from 3.12.1 to 3.15.0.
Project optional deps
pyproject.toml
Update h5py from 3.14.0 to 3.15.0 across optional dependency groups (cache, cluster, all).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • h5py - 3.13.0 #593 — Similar edits to h5py entries in environment and pyproject files; likely related dependency-alignment change.

Poem

A whisk of paws, a bump so spry,
I hop the version, watch it fly,
From CI burrows to Binder sky,
Tests twitch their noses, give a try.
Thump-thump—h5py leaps high! 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title “Update h5py to 3.15.0” directly and succinctly captures the core change in the pull request, which is bumping the h5py dependency to version 3.15.0 across multiple configuration files without introducing unrelated modifications.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch h5py

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ffcfea7 and 5bb417e.

📒 Files selected for processing (2)
  • .ci_support/environment-integration.yml (1 hunks)
  • binder/environment.yml (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • binder/environment.yml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (6)
  • GitHub Check: unittest_win
  • GitHub Check: unittest_openmpi (ubuntu-22.04-arm, 3.13)
  • GitHub Check: unittest_openmpi (macos-latest, 3.13)
  • GitHub Check: unittest_mpich (ubuntu-22.04-arm, 3.13)
  • GitHub Check: unittest_mpich (macos-latest, 3.13)
  • GitHub Check: minimal
🔇 Additional comments (1)
.ci_support/environment-integration.yml (1)

4-4: h5py version bump looks good

Pin updated cleanly and remains aligned with the rest of the environment spec.


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.

@codecov
Copy link

codecov bot commented Oct 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.16%. Comparing base (5b25827) to head (5bb417e).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #843   +/-   ##
=======================================
  Coverage   98.16%   98.16%           
=======================================
  Files          34       34           
  Lines        1688     1688           
=======================================
  Hits         1657     1657           
  Misses         31       31           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jan-janssen jan-janssen merged commit 32943d6 into main Oct 15, 2025
62 of 63 checks passed
@jan-janssen jan-janssen deleted the h5py branch October 15, 2025 06:23
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