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.

Sign up for GitHub

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

Jump to bottom

np.datetime64 vs pd.Timestamp #12

Closed
mrocklin opened this issue Nov 10, 2016 · 1 comment
Closed

np.datetime64 vs pd.Timestamp #12

mrocklin opened this issue Nov 10, 2016 · 1 comment

Comments

@mrocklin
Copy link
Member

It appears that we convert to datetime64s. Is this intentional? Do we roundtrip cleanly to pandas timestamps with timezones?
@martindurant
Copy link
Member

Can we make a test for this?

@mrocklin mrocklin mentioned this issue Nov 14, 2016
Merged
martindurant pushed a commit to martindurant/fastparquet that referenced this issue Nov 14, 2016
Warn on timestamp coersion
ddd7935 
Fixes dask#12
@martindurant martindurant mentioned this issue Nov 14, 2016
Merged
martindurant added a commit that referenced this issue Nov 14, 2016
@martindurant
Warn on timestamp coersion (#19)
e81e193 
Fixes #12
martindurant added a commit that referenced this issue Nov 15, 2016
@martindurant
More docs (#20)
d913c64 
* Warn on timestamp coersion

Fixes #12

* Flesh out details and a little dev doc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mrocklin @martindurant