Skip to content

fix(deps): update tsam bounds after yanked releases#622

Merged
FBumann merged 3 commits intomainfrom
fix/tsam-yanked-versions
Mar 17, 2026
Merged

fix(deps): update tsam bounds after yanked releases#622
FBumann merged 3 commits intomainfrom
fix/tsam-yanked-versions

Conversation

@FBumann
Copy link
Member

@FBumann FBumann commented Mar 17, 2026

Summary

  • tsam 3.0.0 and 3.1.0 were yanked from PyPI
  • Updated full bounds: >= 3.1.1, < 4 (was >= 3.0.0, < 4, != 3.1.0)
  • Updated dev pin: ==3.1.2 (was ==3.0.0)
  • All 258 clustering tests pass on tsam 3.1.2

Test plan

  • tests/test_clustering/ — 245 tests passed
  • tests/test_math/test_clustering.py — 13 tests passed

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated dependency version constraints for improved compatibility and development consistency.

tsam 3.0.0 and 3.1.0 were yanked from PyPI. Update:
- full: >= 3.1.1, < 4 (was >= 3.0.0, < 4, != 3.1.0)
- dev: ==3.1.2 (was ==3.0.0)

All 258 clustering tests pass on 3.1.2.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 17, 2026

📝 Walkthrough

Walkthrough

This pull request updates the tsam time-series library dependency constraints in both CHANGELOG.md and pyproject.toml. The version bounds change from >= 3.0.0, < 4, != 3.1.0 to >= 3.1.1, < 4, and the development pin updates to 3.1.2, reflecting that tsam versions 3.0.0 and 3.1.0 have been yanked from PyPI.

Changes

Cohort / File(s) Summary
Dependency documentation
CHANGELOG.md, pyproject.toml
Updated tsam version constraints from >= 3.0.0, < 4, != 3.1.0 to >= 3.1.1, < 4 in full feature set, and dev pin from 3.0.0 to 3.1.2. CHANGELOG documents that tsam 3.0.0 and 3.1.0 were yanked from PyPI.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A hop through the versions we go,
Dependencies dancing, old yanks steal the show,
Three-one-one-one, the new minimum height,
Tsam's bouncing forward—a release done right! 🌟

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: updating tsam dependency bounds in response to yanked releases.
Description check ✅ Passed The description is mostly complete with a clear summary and test plan, but does not follow the provided template structure with sections like 'Type of Change' or 'Related Issues'.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/tsam-yanked-versions
📝 Coding Plan
  • Generate coding plan for human review comments

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.

FBumann and others added 2 commits March 17, 2026 12:57
tsam 3.0.0 and 3.1.0 were yanked from PyPI. Update:
- full: >= 3.1.1, < 4 (was >= 3.0.0, < 4, != 3.1.0)
- dev: ==3.1.2 (was ==3.0.0)

Fold dependency note into 6.1.0 release notes.

All 258 clustering tests pass on 3.1.2.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
pyproject.toml (1)

66-66: Add CI validation for the minimum supported tsam version.

full permits tsam >= 3.1.1, but CI only validates against tsam==3.1.2 (via .[dev]). Either add a test matrix entry for tsam==3.1.1 or raise the lower bound to >=3.1.2 to match tested coverage.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@pyproject.toml` at line 66, The dependency line "tsam >= 3.1.1, < 4" in
pyproject.toml declares 3.1.1 as the minimum but CI only tests with tsam==3.1.2
via the dev extras; either add a CI matrix entry that runs the test suite
against tsam==3.1.1 or change the pyproject.toml constraint to "tsam >= 3.1.2, <
4" so the declared minimum matches what CI validates—update the dependency in
pyproject.toml or extend the CI test matrix accordingly and rerun CI to confirm
consistency.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@pyproject.toml`:
- Line 66: The dependency line "tsam >= 3.1.1, < 4" in pyproject.toml declares
3.1.1 as the minimum but CI only tests with tsam==3.1.2 via the dev extras;
either add a CI matrix entry that runs the test suite against tsam==3.1.1 or
change the pyproject.toml constraint to "tsam >= 3.1.2, < 4" so the declared
minimum matches what CI validates—update the dependency in pyproject.toml or
extend the CI test matrix accordingly and rerun CI to confirm consistency.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 8e60ba6c-431a-43c5-a6b5-1217039bf04f

📥 Commits

Reviewing files that changed from the base of the PR and between f1209b8 and 1e706f7.

📒 Files selected for processing (2)
  • CHANGELOG.md
  • pyproject.toml

@FBumann FBumann merged commit 1a8f199 into main Mar 17, 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.

1 participant