Skip to content

deps: unpin pydantic, bump wandb 0.21.0 -> 0.22.3#935

Open
ocg-goodfire wants to merge 1 commit into
feature/jaxfrom
bridge/task-remove-pydantic-pin
Open

deps: unpin pydantic, bump wandb 0.21.0 -> 0.22.3#935
ocg-goodfire wants to merge 1 commit into
feature/jaxfrom
bridge/task-remove-pydantic-pin

Conversation

@ocg-goodfire

Copy link
Copy Markdown
Collaborator

Description

Removes the pydantic<2.12 pin (back to the pre-pin pydantic>=2) and bumps wandb==0.21.0wandb==0.22.3. Lockfile now resolves pydantic 2.13.4 (current latest).

The wandb bump is load-bearing: the warnings the pin silenced come from wandb's own use of pydantic Field(), fixed upstream in wandb/wandb#10651 and first released in v0.22.3 — our pinned 0.21.0 predates it, so removing the pydantic pin alone would bring the spam back. 0.22.3 is the minimal fixed release (deliberately not jumping to current 0.28.0; further wandb bumps are a separate concern).

Related Issue

Closes #233 (and re-resolves #221 without the pin)

Motivation and Context

The pin was always meant to be temporary ("remove once wandb fixes it") — wandb shipped the fix in Oct 2025.

How Has This Been Tested?

  • Isolated-venv matrix with pydantic 2.13.4: wandb 0.21.0 → 6× UnsupportedFieldAttributeWarning on import; wandb 0.22.3 / 0.28.0 → zero warnings.
  • import wandb + param_decomp.configs + param_decomp.built_run clean under warnings.simplefilter("error") with the new lock.
  • Full non-slow suite on the new lock: 458 passed, 7 skipped, 11 xfailed (SLURM job 170744).
  • make type (basedpyright): 0 errors — the pinned basedpyright<1.32.0 is still fine with wandb 0.22.3 (that pin is a separate pyright issue, left untouched).

Does this PR introduce a breaking change?

No.

🤖 Generated with Claude Code

The pydantic pin (#232) only silenced wandb's
UnsupportedFieldAttributeWarning spam under pydantic 2.12+ (#221).
wandb fixed that in wandb/wandb#10651, first released in v0.22.3, so the
pin can go once wandb is on a fixed release. Locks pydantic at 2.13.4.

Verified: importing wandb + the config schema is clean under
warnings-as-errors with wandb 0.22.3 / pydantic 2.13.4 (wandb 0.21.0
emits 6 warnings), and the full non-slow suite passes (458 passed).

Closes #233

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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