Fix solemodels interface

Author: giopaglia

ext/DecisionTreeExt.jl

@@ -54,14 +54,14 @@ end
 
 function SoleModels.solemodel(
     model::DT.Ensemble,
+    args...;
     classlabels = nothing,
     featurenames = nothing,
-    args...;
     keep_condensed = true,
     kwargs...
 )
     if isnothing(classlabels)
-        error("Please, provide classlabels argument, as in solemodel(forest, classlabels; kwargs...). If your forest was trained via MLJ, use `classlabels = (mach).fitresult[2][sortperm((mach).fitresult[3])]`.")
+        error("Please, provide classlabels argument, as in solemodel(forest; classlabels = classlabels, kwargs...). If your forest was trained via MLJ, use `classlabels = (mach).fitresult[2][sortperm((mach).fitresult[3])]`. Also consider providing `featurenames = report(mach).features`.")
     end
     if keep_condensed
         info = (;

test/DecisionTreeExt/forest.jl

@@ -41,7 +41,8 @@ fit!(mach)
 classnames = (mach).fitresult[2]
 classnames = classnames[sortperm((mach).fitresult[3])]
 featurenames = report(mach).features
-solem = solemodel(fitted_params(mach).forest, classnames, featurenames)
+solem = solemodel(fitted_params(mach).forest; classnames, featurenames)
+solem = solemodel(fitted_params(mach).forest; classnames, featurenames, keep_condensed = false)
 
 @test SoleData.scalarlogiset(X_test; allow_propositional = true) isa PropositionalLogiset