Benjamin Bossan

_Linting errors seem to be unrelated to this PR_

> Syncing with `main` should help. The linting error was recently fixed in #24065 I think since this PR is against `sample-props`, the `sample-props` branch needs to be synced first,...

I updated but the linting still fails for unrelated reasons. It also fails on the `sample-props` branch itself because of what looks like changes introduced by the multioutput PR: https://dev.azure.com/scikit-learn/scikit-learn/_build/results?buildId=45465&view=logs&j=32e2e1bb-a28f-5b18-6cfc-3f01273f5609&t=8a54543f-0728-5134-6642-bedd98e03dd0...

I didn't review line by line but since the linter is green now, this should be good to merge.

> I just ran isort on the files, no manual change. isort of expected that (okay, I'll see myself out)

Let me preface this by admitting that I have very little experience with keras but here are my thoughts: As a user, when I would see a callback like this,...

@merveenoyan I added a feature ([this PR](https://github.com/skorch-dev/skorch/pull/881)) on skorch to save model checkpoints on HF Hub. I took a different approach there which works with _existing callbacks_ instead of writing...

Hi Srimal, Thank you for your contribution. I did not look at your code, but it seems to me that what you described is already possible with the current version...

I'm not too sure what you mean, do you want to use `plot_conv_weight` with a CNN that takes colored images? Because I think that should already work. In general, if...

I don't think plotting the weights in color will give you much additional insights. Why you get black images I can't tell but you should look at whether the values...