Skip to content

Conversation

@WeichenXu123
Copy link
Contributor

@WeichenXu123 WeichenXu123 commented Aug 27, 2016

What changes were proposed in this pull request?

Add treeAggregateDepth parameter for AFTSurvivalRegression to keep consistent with LiR/LoR.

How was this patch tested?

Existing tests.

@SparkQA
Copy link

SparkQA commented Aug 27, 2016

Test build #64536 has finished for PR 14851 at commit 7ef9d8b.

  • This patch fails Scala style tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Aug 28, 2016

Test build #64542 has finished for PR 14851 at commit 2eafb77.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Sep 21, 2016

Test build #65722 has finished for PR 14851 at commit ae37bfd.

  • This patch fails MiMa tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@WeichenXu123 WeichenXu123 force-pushed the add_treeAggregate_param_for_survival_regression branch from ae37bfd to 378079d Compare September 22, 2016 01:44
@SparkQA
Copy link

SparkQA commented Sep 22, 2016

Test build #65753 has finished for PR 14851 at commit 378079d.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@yanboliang
Copy link
Contributor

LGTM, merged into master. Thanks!

@asfgit asfgit closed this in 72d9fba Sep 22, 2016
@WeichenXu123 WeichenXu123 deleted the add_treeAggregate_param_for_survival_regression branch September 22, 2016 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants