Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TreeEnsembleRegressor/Classifier have been deprecated in favour of TreeEnsemble #1160

Open
khoover opened this issue Jan 23, 2025 · 1 comment

Comments

@khoover
Copy link

khoover commented Jan 23, 2025

The decision tree converters currently default to using the deprecated operator types, even in op versions where they've both been long-deprecated (didn't realize it was under the ml ops, not the regular ones). I believe the fix would be to use a default of op_type=None, plus if op_type is None: op_type = "TreeEnsemble" if op_version >= 5 else "TreeEnsemble...". There might be more that needs to happen for the node attributes, I haven't looked at the differences between the old op types and the new one too close.

@khoover
Copy link
Author

khoover commented Jan 23, 2025

Related to #1051

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

No branches or pull requests

1 participant