-
Notifications
You must be signed in to change notification settings - Fork 335
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
Tf catvars from tlapusan #265
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code looks good to me, besides one print statement. I don't see text axis tick labels in either of the demo notebooks though? sex_label
is still 0 or 1.
We have an updated the notebooks yet but I do have tutorial I'm working on that uses categoricals. Take a look at I need to fix |
…: can't show avg lines as we can for numeric X values
…ered set: can't show avg lines as we can for numeric X values" This reverts commit 6bb0806.
Signed-off-by: Terence Parr <parrt@antlr.org>
…into tf-catvars-from-tlapusan
Signed-off-by: Terence Parr <parrt@antlr.org>
…one value not multiple; normalize to list Signed-off-by: Terence Parr <parrt@antlr.org>
Signed-off-by: Terence Parr <parrt@antlr.org>
…into tf-catvars-from-tlapusan
Signed-off-by: Terence Parr <parrt@antlr.org>
Starting from @tlapusan, raising exceptions when categorical variable's are used in feature space. I just don't know how to display that at the moment. Popping the version to 2.2.0 to incorporate categorical vars.
Derived from #263
We still need to test the decision notes more, such as when there are lots of levels in the catvars.