Skip to content
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

ENH: Fill out more pandas backend operations #1423

Closed
wants to merge 47 commits into from

Conversation

cpcloud
Copy link
Member

@cpcloud cpcloud commented Apr 14, 2018

Also fixes an xfailed test because of postgres

@cpcloud cpcloud added tests Issues or PRs related to tests postgres The PostgreSQL backend expressions Issues or PRs related to the expression API pandas The pandas backend labels Apr 14, 2018
@cpcloud cpcloud added this to the 0.14 milestone Apr 14, 2018
@cpcloud cpcloud self-assigned this Apr 14, 2018
@cpcloud cpcloud changed the title ENH: Add scalar math functions to pandas ENH: Fill out more pandas backend operations Apr 14, 2018
@cpcloud cpcloud force-pushed the pandas-operations branch 4 times, most recently from 920be04 to 78afb7e Compare April 16, 2018 14:34
@cpcloud cpcloud force-pushed the pandas-operations branch 2 times, most recently from cabf29f to bfc4622 Compare May 13, 2018 20:50
@@ -164,7 +181,11 @@ def schema_from_table(table, schema=None):
if name in schema:
dtype = dt.dtype(schema[name])
else:
dtype = dt.dtype(column.type, nullable=column.nullable)
dtype = dt.dtype(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ohh, this is tricky!

Copy link
Member

@kszucs kszucs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like conda would say:

I just wanted to let you know that I reviewed your changes and found it was in an excellent condition.

Nice to see that all tests are gaining weight.

@cpcloud
Copy link
Member Author

cpcloud commented May 15, 2018

@kszucs Thanks for the review, yeah the all tests are definitely exposing edge cases.

@cpcloud cpcloud closed this in 762b98d May 18, 2018
@cpcloud cpcloud deleted the pandas-operations branch May 18, 2018 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
expressions Issues or PRs related to the expression API pandas The pandas backend postgres The PostgreSQL backend tests Issues or PRs related to tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants