search

parrt / dtreeviz

A python library for decision tree visualization and model interpretation.
MIT License
2.94k stars 331 forks source link

fix #304 issue #306

Closed MohammedAlsayed closed 1 year ago

MohammedAlsayed commented 1 year ago

Adding a more informative message when target values are not type int.

tlapusan commented 1 year ago

@MohammedAlsayed thanks for the PR. I just checked your changes () and they are working very good. Checked also to don't have some side effects on the other visualisation methods.

@parrt I will merge this PR and if you consider you can put it into master for a new minor release.

parrt commented 1 year ago

Well, probably not worth doing a release for just one commit...or do we have other stuff to go out?

tlapusan commented 1 year ago

Well, probably not worth doing a release for just one commit...or do we have other stuff to go out?

from what I know only this fix.