Skip to content

Use a mixin method to init the callback_context#32

Merged
jeremiedbb merged 9 commits intojeremiedbb:base_callbacks_2from
FrancoisPgm:init_callback_context
Feb 18, 2026
Merged

Use a mixin method to init the callback_context#32
jeremiedbb merged 9 commits intojeremiedbb:base_callbacks_2from
FrancoisPgm:init_callback_context

Conversation

@FrancoisPgm
Copy link
Copy Markdown

Move the init of the callback context from the fit decorator back to a method of the mixin, called in fit.

@jeremiedbb jeremiedbb added this to Labs Feb 15, 2026
@jeremiedbb jeremiedbb moved this to In progress in Labs Feb 15, 2026
FrancoisPgm and others added 3 commits February 17, 2026 12:03
Co-authored-by: Jérémie du Boisberranger <jeremie@probabl.ai>
Co-authored-by: Jérémie du Boisberranger <jeremie@probabl.ai>
…, add task_id arg to init_callback, add test for est without subtask
@jeremiedbb
Copy link
Copy Markdown
Owner

The example of implementation in the big comment above the callback context class needs to be updated with the method.

@FrancoisPgm
Copy link
Copy Markdown
Author

The example of implementation in the big comment above the callback context class needs to be updated with the method.

Ah yes good catch.

Copy link
Copy Markdown
Owner

@jeremiedbb jeremiedbb left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks !

@jeremiedbb jeremiedbb merged commit a09a406 into jeremiedbb:base_callbacks_2 Feb 18, 2026
10 of 11 checks passed
@github-project-automation github-project-automation bot moved this from In progress to Done in Labs Feb 18, 2026
@FrancoisPgm FrancoisPgm deleted the init_callback_context branch February 19, 2026 11:01
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