Skip to content

Conversation

ablaom
Copy link
Member

@ablaom ablaom commented Jul 13, 2025

In this PR we:

Add InteractiveUtils standard library as dependency

This allows the implemented_methods trait fallback to be immediately defined in this package (like all the other trait fallbacks) rather than relying on the light/full interface mechanism. The previous exceptional behaviour has been causing me issues in a revamp of the MLJ Model Registry management tools.

(in the light/full switch, some methods do not get sensible overloadings until MLJBase is loaded.)

Of course for users there is no change in behaviour, for MLJBase is always loaded, in which case my changes here have no detectable impact. (In my use case for registry management, I want to inspect trait values without having to load all of MLJBase.)

Add a new tags trait

The name is owned by StatisticalTraits, to which this was recently added. The StatisticalTraits fallback of "", suffices. This trait is intended to have values like ["regression", "gradient descent"] and to be overloaded by model interfaces, to remove the burden of maintaining this database at MLJ docs.

@ablaom
Copy link
Member Author

ablaom commented Jul 13, 2025

To do:

Check locally tests for the following downstream packages:

  • MLJBase
  • MLJTuning
  • MLJIteration
  • MLJEnsembles
  • MLJBalancing Tests currently failing for unrelated reason.
  • FeatureSelection
  • MLJ (including the integration tests)

@ablaom ablaom marked this pull request as draft July 13, 2025 04:02
@ablaom ablaom marked this pull request as ready for review July 13, 2025 10:25
@ablaom ablaom merged commit cfc927b into dev Jul 17, 2025
3 checks passed
This was referenced Jul 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant