Anthony Blaom, PhD
Anthony Blaom, PhD
Agreed, this could do with some improvement. The `range` extensions are provided by MLJBase, so transferring this issue to there. `unit` and `origin` are used when fitting pdfs to a...
> I also think that scale could be improved by using an ADT, like you'll find in MLStyle.jl; Not sure I understand the referenced to MLStyle.jl. Can you be more...
Thanks for this investigation. I find it hard to imagine a use-case where the performance improvement of tuples is worth a breaking change. No objection to *allowing* tuples. PR welcome.
I can see where this is coming from but must admit this never confused me, as an outside user. I do like that `order!` is a verb while `levels!` is...
> The question becomes do you prefer that we: > Make it link to the [For Developer](https://github.com/JuliaAI/DataScienceTutorials.jl?tab=readme-ov-file#-for-developers) > section in the README. > OR replicate that in a new page...
Thanks indeed for this PR. Can you please look at the CI fails? For example, we have: ``` collate: Error During Test at /home/runner/work/MLJFlux.jl/MLJFlux.jl/test/core.jl:25 [648](https://github.com/FluxML/MLJFlux.jl/actions/runs/10738332003/job/29781691377?pr=272#step:6:651) Test threw exception [649](https://github.com/FluxML/MLJFlux.jl/actions/runs/10738332003/job/29781691377?pr=272#step:6:652) Expression:...
@tiemvanderdeure If you can fix the fails here, I can sort out the conflict.
Okay @tiemvanderdeure, thanks. Turns out conflict resolution is a bit more complicated than anticipated. So I'm waiting on resolution of the following before trying again: - [x] #273
@tiemvanderdeure Your PR has now been merged and is part of 0.6, just released. Thanks for you patience as we sorted out the conflicts.
Thanks for working on an MLJ interface! Any chance you could add an interface test, as [here](https://github.com/JuliaAI/MLJXGBoostInterface.jl/blob/402861a70fb532f8eddec77dc9d40c6c515d6668/test/runtests.jl#L255)? You'll need MLJTestInterface in your `[extras]`.