-
Notifications
You must be signed in to change notification settings - Fork 57
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
TODOs in test_gbt_formulation #157
Comments
Removed these (and the commented test) in my fork, will close on integration. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
A couple of TODO items in this test file:
# TODO @cog-imperial: fix below?:
- unclear what this is referring to# TODO @cog-imperial: did we remove categorical variables intentionally?
, followed by a test for a model with categorical variables. We should make a decision on whether to remove this and either restore or remove the test.The text was updated successfully, but these errors were encountered: