Remove LightGBMBooster's mergeModels function#316
Remove LightGBMBooster's mergeModels function#316mhamilton723 merged 2 commits intomicrosoft:masterfrom
Conversation
|
@imatiach-msft @mhamilton723 know how to make the build-pr happy? :) |
|
@ywskycn yes, sorry, there was a major refactor that @mhamilton723 recently did to use Azure Databricks cluster for our tests instead of HDInsight and now it looks like the e2e builds are getting stuck, he is looking into it. The PR will be checked in as soon as the build issue is resolved, sorry for the inconvenience. |
|
Sorry @ywskycn this is not your issue and I will check this in as soon as I iron out the build kinks, hopefully by tomorrow. If this is a blocker please let us know |
|
@ywskycn could you quickly rebase your changes on latest master so we can push it through the new build? Thanks! |
|
MMLSpark 0.12.dev20+2.ge0d7a7fThis is a build for Github PR #316, changes:
|
Discussed in #315, the merge function doesn't need anymore. Would better to remove it.