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

Removed time_index_reduce parameter #344

Merged
merged 1 commit into from Dec 12, 2018

Conversation

Projects
None yet
2 participants
@jeff-hernandez
Copy link
Collaborator

jeff-hernandez commented Dec 12, 2018

I finished the adjustment. Let me know if anything needs modification. Thanks.

@jeff-hernandez jeff-hernandez force-pushed the jeff-hernandez:master branch from 26fb844 to 7c93194 Dec 12, 2018

@kmax12

This comment has been minimized.

Copy link
Member

kmax12 commented Dec 12, 2018

Looks good. Merging

@kmax12 kmax12 self-requested a review Dec 12, 2018

@kmax12

kmax12 approved these changes Dec 12, 2018

@kmax12 kmax12 changed the title Fixed #301 - Removed time_index_reduce parameter Removed time_index_reduce parameter Dec 12, 2018

@kmax12 kmax12 merged commit 9ed8ca9 into Featuretools:master Dec 12, 2018

1 check passed

license/cla Contributor License Agreement is signed.
Details

@rwedge rwedge referenced this pull request Dec 17, 2018

Merged

v0.5.0 #351

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment