Skip to content

[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

Closed
wants to merge 35 commits into from
Closed

Conversation

jblackburne
Copy link
Contributor

Creating this PR to track down Appveyor failures in #4899.

Will close once they are figured out.

…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.
…rectly with ensembles. Refactored the categorical transformations into functions.
@nelson-liu
Copy link
Contributor

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.

@jblackburne
Copy link
Contributor Author

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.

@nelson-liu
Copy link
Contributor

it's actually free for open source projects (i.e. non-private repos on github)

@jblackburne
Copy link
Contributor Author

Ok I was able to set up my own account. Sorry for the line noise.

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.

2 participants