Skip to content

Conversation

@joellidin
Copy link
Collaborator

@joellidin joellidin commented Nov 12, 2025

  • (hparams) Update checkpoint init to v2.1.15
  • Bump run version

Description

Related Issue(s)

  • Closes #[issue number]

Type of Change

  • Feature (adding new functionality)
  • Fix (resolving a bug or issue)
  • Docs (documentation updates)
  • Refactor (code changes that don't affect functionality)
  • Maintenance (dependency updates or other maintenance)
  • Tests (adding or improving tests)
  • Breaking change (fix or feature with incompatible API changes)
  • Other: _____

Branch Naming

  • My branch follows the project's naming convention (e.g., feature/add-new-capability)

Commit Messages

  • My commits are small, atomic, and have proper commit messages
  • Commit messages are in imperative mood with a capitalized summary under 50 chars

Code Quality

  • I've performed a self-review of my code
  • I've added appropriate docstrings following the project's conventions
  • I've added proper logging where necessary (without trailing periods)
  • I've applied linting and formatting with Ruff
  • My code generates no new warnings

Testing

  • I've added tests for new functionality or bug fixes
  • All tests pass locally with my changes
  • Test coverage has not decreased

Documentation

  • I've updated documentation to reflect my changes
  • I've updated comments in hard-to-understand areas

If this is a breaking change

Screenshots/Examples

Additional Notes

Summary by CodeRabbit

  • Chores
    • Version updated to 2.1.16 with checkpoint initialization updates.

Update checkpoint initialization parameters to align with the latest
version release.

- Bump checkpoint_init_version from 2.1.14 to 2.1.15
- Update checkpoint_init_window from 59518 to 59647
@coderabbitai
Copy link

coderabbitai bot commented Nov 12, 2025

Walkthrough

Two version-related updates: hyperparameter checkpoint metadata incremented from version 2.1.14/window 59518 to 2.1.15/59647 in hparams.json, and package version incremented to 2.1.16 in src/tplr/init.py.

Changes

Cohort / File(s) Summary
Version and checkpoint metadata updates
hparams/hparams.json, src/tplr/__init__.py
Updated checkpoint_init_version to 2.1.15, checkpoint_init_window to 59647, and bumped package version to 2.1.16

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 A hop, a skip, a version bump so true,
From 2.1.15 the checkpoints grew,
Now 2.1.16 claims the stage,
Each digit marks a rabbit's page! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is largely incomplete, with only changelog-style bullet points provided and all template sections left blank or unchecked, including missing explanation of the 'why' and version bump rationale. Fill in the Description section with context about why these version updates are needed, select appropriate Type of Change option(s), and verify other relevant checklist items are addressed.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'hparams/update bootstrap version' directly relates to the main change: updating hparams/hparams.json checkpoint parameters and bumping the package version.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch hparams/update-bootstrap-version

📜 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 746e680 and 4f636a0.

📒 Files selected for processing (2)
  • hparams/hparams.json (1 hunks)
  • src/tplr/__init__.py (1 hunks)
⏰ 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). (2)
  • GitHub Check: test (3.12)
  • GitHub Check: test (3.11)
🔇 Additional comments (2)
src/tplr/__init__.py (1)

23-23: LGTM!

The package version bump from 2.1.15 to 2.1.16 is straightforward and aligns with the PR objective.

hparams/hparams.json (1)

46-47: Verify version offset between package and checkpoint versions.

The package version (2.1.16 in src/tplr/__init__.py) is now ahead of the checkpoint_init_version (2.1.15). Confirm this one-version offset is intentional and reflects the expected initialization versioning strategy.


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 Nov 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev     #654   +/-   ##
=======================================
  Coverage   57.91%   57.91%           
=======================================
  Files          27       27           
  Lines        4890     4890           
=======================================
  Hits         2832     2832           
  Misses       2058     2058           
Files with missing lines Coverage Δ
src/tplr/__init__.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@joellidin joellidin merged commit c449fad into dev Nov 12, 2025
7 of 8 checks passed
@joellidin joellidin deleted the hparams/update-bootstrap-version branch November 12, 2025 06:21
@coderabbitai coderabbitai bot mentioned this pull request Nov 12, 2025
21 tasks
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