Victor Negîrneac
Victor Negîrneac
@mwouts Thank you very much, it seems to solve the problem indeed!
I believe this has been solved now, closing
@MuhammedHasan I just noticed your changes incorporated also the fix in #60 that you might have forgotten to mention. See my suggestion in https://github.com/jmschrei/tfmodisco-lite/pull/60#issuecomment-3224149495 regarding fixing the default argument value...
Old issue but maybe it will still be useful to someone: In the code you provided you seem to be reusing one of the non-linear nn.Module instances more than once...
@jmschrei @bytewife I am wondering if you spot this PR? and if there is any relevant problems that this bug might create I suggest to additionally change the default argument...
I understand, thank you for the reply 🙂
For future readers, see my comments on these two other issues: https://github.com/shap/shap/issues/3938#issuecomment-3205824703 (this one seem to apply here) https://github.com/shap/shap/issues/3735#issuecomment-3205861143 (this is good to keep in mind)
Hi Mark, I have been working on making use of `jupytext --paired-paths` in Jupytext Sync VS Code extension and the topic of the exit codes came up again and I...
Quick update, I used the internal python functions for the VS Code extension. It is actually better because I can output JSON from Python and load it again in JS...
Workaround: set a keyboard shortcut for commenting cells (command id `notebook.commentSelectedCells`) to the `R` key