Github user sethah commented on the pull request:
https://github.com/apache/spark/pull/12050#issuecomment-203156556
One concern I had was that `evaluateEachIteration`,
`computeInitialPredictionError`, and `updatePredictionError` are implemented in
ML _and_ MLlib. They need to remain unless we break the API. I don't think
there's an easy way to have them wrap the implementations in ML because
converting MLlib GBTs to ML GBTs requires information about categorical
features which is not stored in the existing MLlib architecture.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
