Anthony Blaom, PhD

Great to have this contribution, thanks. A cursory look at the code suggests this looks good. Can you please bring the the hyperparmeter struct docstring up to [spec](https://juliaai.github.io/MLJModelInterface.jl/dev/document_strings/)? @ablaom To...

Waiting for: - [x] https://github.com/ml-unito/PartitionedLS.jl/issues/4 - [ ] Feedback on https://github.com/ml-unito/PartitionedLS.jl/pull/7

Thanks @boborbt for that. Would you consider updating the docstring as indicated above?

Okay, I see this is done, many thanks! Please see my proposed update, https://github.com/ml-unito/PartitionedLS.jl/pull/7.

> I've tried to follow the spec, but I was not able to use the $(MLJModelInterface.doc_header(...)) call. For some reason if I include it precompiling the package fails asserting that...

Sorry, I posted an comment but must have forgotten to click "comment". I'm waiting for you to tag a new release to make the doc strings live. The registration process...

I suggest you only add a precompile load for your "native" API. If your native API depends on MLJBase to work, then we ought to be able to fix that....

https://alan-turing-institute.github.io/MLJ.jl/dev/models/PartLS_PartitionedLS/

Thanks @boborbt for your patience with the model adding process. It was helpful to have the prompt responses to my suggestions to get this sorted quickly. Congratulations again on the...

Thanks for reporting. Looks like a package management issue. Some packages in your existing environment are in conflict with the package compatibility requirements of MLJTutorials.jl. This is not fatal, because...