Can't access the number of tree nodes

[PatrickZENG-NTU]

Thank you for providing this useful package of the oblique trees. It's really useful to some academic users like me who currently try to develop a new multivariate tree model. Although you provide the classification results of the oblique tree, it is a little pity that I can't access the number of tree nodes. The number of tree nodes is a critical feature for the comparative studies of tree models. So it will be wonderful for academic users like me if you can provide the information on the number of tree nodes in future versions.

[AndriyMulyar]

For this you'll have to look into the accompanying Cython code that interfaces the original C based OC1 implementation. The number of nodes should be readily accessible as a global variable to extract out. I unfortunately will not have the bandwidth to do this in the near future. I'll happily review a PR if you implement this! Thanks!

…

On Mon, Nov 16, 2020, 9:01 AM PatrickZENG-NTU ***@***.***> wrote: Thank you for providing this useful package of the oblique trees. It's really useful to some academic users like me who currently try to develop a new multivariate tree model. Although you provide the classification results of the oblique tree, it is a little pity that I can't access the number of tree nodes. The number of tree nodes is a critical feature for the comparative studies of tree models. So it will be wonderful for academic users like me if you can provide the information on the number of tree nodes in future versions. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#5>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADJ4TBUVRQDQTQHPLQHU5ZLSQEWELANCNFSM4TXGMCBA> .

[PatrickZENG-NTU]

Thanks for your quick reply. This is a good suggestion to solve this problem. I will try to implement it. If I successfully resolve this issue, I will send you a message about it.

[ZiruiYan]

Hi, I meet the same problem, have you solved it? @PatrickZENG-NTU