Skip to content

fix WandbLoggerConfig include additional kwargs in pydantic schema - #3469

Draft
deruyter92 wants to merge 1 commit into
mainfrom
jaap/fix-wandb-logger-config
Draft

fix WandbLoggerConfig include additional kwargs in pydantic schema#3469
deruyter92 wants to merge 1 commit into
mainfrom
jaap/fix-wandb-logger-config

Conversation

@deruyter92

Copy link
Copy Markdown
Collaborator

The Weights and Biases logger accepts a variety of extra arguments, such as notes, tags, etc. These were mediated through wandb_kwargs which could be defined at top-level. However since the introduction of the stricter pydantic schemas these top-level arguments with extra="forbid" would raise a ValidationError.

This PR:

  • fixes a dead wandb_kwargs field in the WandBloggerConfig (was never passed correctly to the logger)
  • adds common top-level fields such as notes, tags, group to the top-level config.

@deruyter92 deruyter92 added this to the v3.1.0 milestone Aug 28, 2026
@deruyter92 deruyter92 added the 3.1 label Aug 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant