-
Notifications
You must be signed in to change notification settings - Fork 390
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
Optional relationals #1292
Optional relationals #1292
Conversation
looks like we need to import pytest |
MAINT: fix _dtype property in Not class. TST: fix tests. TST: clean up imports. TST: fix imports
Yup, sorry. I removed that mistakenly. It's back now. |
b91f6ba
to
a40d1a7
Compare
Can you add a new entry to the whatsnew? |
Sure - which file to I add it to? For the latest version, there's BASE, LOCAL, and REMOTE. I think those were somehow created automatically, and I'm not sure what to do with them. |
docs/source/whatsnew/0.9.0.txt |
Done. |
DOC: fix whatsnew entry.
46cd018
to
4efd105
Compare
def test_relations_are_boolean(): | ||
assert Gt(x, y).schema == dshape('bool') | ||
@pytest.mark.parametrize('relation_type', Relational.__subclasses__()) | ||
@pytest.mark.parametrize('lhs,rhs', product((x, y, d, e, f), repeat=2)) |
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.
neat! i didn't realize you could chain these
i'm going to resolve the merge conflicts here and merge on passing |
Modify Arithmetic classes so that operations with optionals result in an optional type and add associated tests.