Open ablaom opened 1 year ago
Similar to https://github.com/OutlierDetectionJL/OutlierDetection.jl/issues/32. It would make sense to propose a default model architecture or raise an informative error if none is provided, I'll fix that in the next release.
Elsewhere models that are not wrappers have a constructor with no arguments that returns an instance with default hyper-parameters. That is DSADDetector()
should just work. This is not in the spec, but probably should be, and it is assumed to be the case in integration tests at MLJTestIntegration.jl. If this does not make sense for these models, then I suggest we not include them in the model registry (wrappers, like TunedModel
and Stack
are not in the registry). Let me know if this is the case.
We would probably have to implement https://github.com/OutlierDetectionJL/OutlierDetectionNetworks.jl/issues/3 before to use something like builders as in MLJFlux, otherwise the shape of the data would have to be known beforehand to propose a network architecture.
Ah, I see. That makes sense. Any objections to (temporarily) deregisteringOutlierDetectionNetwork models? I can add the package to this list.
No objections, we can remove again once the issue is closed.
There is also a problem with ESADDetector.
Project: