Skip to content

False warning#71

Open
JackStat wants to merge 2 commits intogbm-developers:masterfrom
JackStat:falseWarning
Open

False warning#71
JackStat wants to merge 2 commits intogbm-developers:masterfrom
JackStat:falseWarning

Conversation

@JackStat
Copy link
Contributor

@JackStat JackStat commented Nov 2, 2015

I went ahead and added some tests for coverage against the false warning that we are seeing. It is only impacting bernoulli distributions but the coverage on the others won't hurt. I don't have any ideas on how to fix the issue but the test should speed up the process a bit. Should help resolve #69

@JackStat
Copy link
Contributor Author

Are there any ideas on what the cause of this issue is?

@pdmetcalfe
Copy link
Member

I'm wondering if we just randomly get down to something separable?

On Wed, 18 Nov 2015, 15:48 Tyler Hunt notifications@github.com wrote:

Are there any ideas on what the cause of this issue is?


Reply to this email directly or view it on GitHub
gbm-developers/gbm#71 (comment).

@JackStat
Copy link
Contributor Author

I get a warning on all of the tests in basic checks so I don't think it is a separation issue. It really seems like a false warning.

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.

False "... terminal node predictions were excessively large..." warning

2 participants

Comments