-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
[WIP] nocats CI testing #7068
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] nocats CI testing #7068
Conversation
…to categorical variables. Replaced the threshold attribute of SplitRecord and Node with SplitValue.
…hat defaults to -1 for each feature (indicating non-categorical feature).
…(). Enforced max n_categories=2**31 for RandomSplitter.
…rating a big list of random numbers for every sample in every node when sending it left or right.
…hat there are no sparse categorical variables, since they're currently not supported.
…ith more than 64 categories.
… to make sure we free the bit cache.
…splits until a non-trivial one is found
…rectly with ensembles. Refactored the categorical transformations into functions.
…identally creates a trivial split.
…resented in the local sample.
…gorical features.
…iate for categorical features.
…n with numpy v1.6. This should be reverted when support for numpy v1.6 is dropped.
…ees, forests, and gradient boosting. Tweaked unit tests to match.
fyi, you can create an appveyor account for yourself, add your fork to it, and then run tests on branches on that fork and view results with your own appveyor account. this has the added benefit of also not having to wait for the queued builds from other PRs to finish before seeing your results. |
Thanks. Looks like free accounts are limited to 14 days though. That's probably not going to work for me, given the limited fraction of my time I can devote to this project. |
it's actually free for open source projects (i.e. non-private repos on github) |
Ok I was able to set up my own account. Sorry for the line noise. |
Creating this PR to track down Appveyor failures in #4899.
Will close once they are figured out.