You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think it would be useful for someone with appropriate expertise to review how the model registry is currently implemented. (I think the user interface is basically fine .) This is part of the code that was written very early when there were many competing priorities and I feel there is room for improvement.
The main complaints I have are:
tedious to update. For example, when adding a new model-providing package, one must artificially force precompilation of MLJModels (see Find a way to load metadata from __init__ #74). And with so many packages, this takes a long while, although this is not too big a deal, at least for now.
A bit fragile: In principle anyone can follow the instructions but in practice small issues arise, which I can fix but which a newcomer might scratch their head over. The process would be hard to automate (using CI, for example) without some improvements.
lacking some flexibility in terms of the kind of metadata that can be serialised. Currently we basically do our own serialization into a toml file, which has worked so far, but for maintenance and extra generality, it might make better sense to use an external package or julia serialiaztion for this. Maybe.
I believe @vollmersj had the very good idea to borrow the approach used in the julia package registry. I did not and still do not have enough familiarity with this system to take this approach but could indeed be best approach. In any case, someone with experience with that code base would presumably be in a very good position to propose a better design.
Maybe Pkg Artifacts could play a role??
The text was updated successfully, but these errors were encountered:
I think it would be useful for someone with appropriate expertise to review how the model registry is currently implemented. (I think the user interface is basically fine .) This is part of the code that was written very early when there were many competing priorities and I feel there is room for improvement.
The main complaints I have are:
tedious to update. For example, when adding a new model-providing package, one must artificially force precompilation of MLJModels (see Find a way to load metadata from __init__ #74). And with so many packages, this takes a long while, although this is not too big a deal, at least for now.
A bit fragile: In principle anyone can follow the instructions but in practice small issues arise, which I can fix but which a newcomer might scratch their head over. The process would be hard to automate (using CI, for example) without some improvements.
lacking some flexibility in terms of the kind of metadata that can be serialised. Currently we basically do our own serialization into a toml file, which has worked so far, but for maintenance and extra generality, it might make better sense to use an external package or julia serialiaztion for this. Maybe.
Some other feedback about the system: #169
I believe @vollmersj had the very good idea to borrow the approach used in the julia package registry. I did not and still do not have enough familiarity with this system to take this approach but could indeed be best approach. In any case, someone with experience with that code base would presumably be in a very good position to propose a better design.
Maybe Pkg Artifacts could play a role??
The text was updated successfully, but these errors were encountered: