parrt / dtreeviz

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

Fix Digraph Graphics #224

Closed mepland closed 1 year ago

mepland commented 1 year ago

dtreeviz_pred_path dtreeviz_pred_path.pdf

mepland commented 1 year ago

Hmm actually looking at it again, the "Prediction 0" text is in the correct spot on the png I save, but not the pdf, so I think that it is fine. I just changed to two leading spaces to match the other labels. The   character doesn't appear to be doing anything in my testing.

Ideally we would just move the edge label positions in the graph itself, but this doesn't appear possible for graphviz:

parrt commented 1 year ago

Ideally we would just move the edge label positions in the graph itself, but this doesn't appear possible for graphviz:

yeah, I had lots of trouble with positioning...

parrt commented 1 year ago

just ran notebooks. looks fine. thanks!