Skip to content

Overhaul documentation for callbacks. - #430

Merged
zoq merged 1 commit into
mlpack:masterfrom
rcurtin:doc-updates
Sep 1, 2025
Merged

Overhaul documentation for callbacks.#430
zoq merged 1 commit into
mlpack:masterfrom
rcurtin:doc-updates

Conversation

@rcurtin

@rcurtin rcurtin commented Aug 27, 2025

Copy link
Copy Markdown
Member

While preparing the answer for #3992, I noticed that the callback documentation didn't point out anything about the StepTaken() function, and the TimerStop callback wasn't documented at all. I overhauled the documentation to organize it a bit better, added documentation for StepTaken() and TimerStop, and reorganized things to hopefully be more useful to users.

@zoq zoq left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks great to me.

@zoq
zoq merged commit 198525c into mlpack:master Sep 1, 2025
1 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants