Skip to content

Python SDK#47

Merged
feast-ci-bot merged 54 commits into
feast-dev:masterfrom
zhilingc:py-sdk
Jan 17, 2019
Merged

Python SDK#47
feast-ci-bot merged 54 commits into
feast-dev:masterfrom
zhilingc:py-sdk

Conversation

@zhilingc
Copy link
Copy Markdown
Collaborator

@zhilingc zhilingc commented Jan 8, 2019

Implementation of the python sdk as outlined in the RFC here: https://github.com/gojek/feast/blob/master/rfcs/0000-python-sdk.md

zhilingc and others added 30 commits January 4, 2019 23:47
Initial commit
a07989e
Add feast resource definitions
0e65567
Add mapping from dtype to valueType
e77ede5
Add utilities for google cloud
5439156
Fix printer to print yaml in order
6c09dfc
Add importer
793fc77
Add client
b3124d2
Update workflow notebook
1edbdc0
Add tests
237b946
Clean up sdk code
1416fe8
Add push to pypi resources
47e4110
Add ValueType and Granularity so that we don't expose internal proto …
4e66d79 
…to client
Scrub sensitve data
7da288e
Add feature_set stub
3b3bcc8
Clean add env, clean up docstrings
0df9c4e
Add create_feature_set implementation
231e803
Move feature set to be a resource
3a9ba39
Add serving retrieval functions
891e3f9
Add FeatureSet and DatasetInfo implementation
7cfad22
Light refactor, fix tests
85b476c
Add checking for different entity in a feature set
5808ccd
Add create_training_dataset api to feast client
09f5a6c
Deduplicate requesting storage specs
6387254
Use TableReference when defining destination table
0bf094d
Fix downloading csv from GCS
5f7bc7b
Add tests and refactoring
ada0335
Refactor and clean move classes locations
5dbeb06
Add license
e30e824
Fix typo
0b3ffee
Move download functionality from DatasetInfo into Client
1c1aaf5
@tims
Copy link
Copy Markdown
Contributor

tims commented Jan 11, 2019

I like the API. Seems like a great start and we can iterate.

One small thing, I think Datastore should be call FeatureStore, but I'm happy if we refactor that in a new issue.

/lgtm

@feast-ci-bot feast-ci-bot removed the lgtm label Jan 12, 2019
@zhilingc
Copy link
Copy Markdown
Collaborator Author

zhilingc commented Jan 12, 2019

I like the API. Seems like a great start and we can iterate.

One small thing, I think Datastore should be call FeatureStore, but I'm happy if we refactor that in a new issue.

/lgtm

Good idea for later versions - we'd have to propagate that name change throughout the system haha.

@woop please review quickstart - I'm still not sure about the dataset since it doesn't showcase our support for real time features, if you can think of a better one I'm open

woop
woop requested changes Jan 12, 2019
View reviewed changes
Comment thread sdk/python/setup.py Outdated
Comment thread sdk/python/feast/sdk/utils/gs_utils.py
Comment thread sdk/python/feast/sdk/utils/bq_util.py
Comment thread sdk/python/examples/quickstart/Quickstart.ipynb
Comment thread sdk/python/examples/quickstart/Quickstart.ipynb Outdated
Comment thread sdk/python/examples/quickstart/Quickstart.ipynb
Comment thread sdk/python/examples/quickstart/Quickstart.ipynb Outdated
Comment thread sdk/python/examples/quickstart/Quickstart.ipynb
Comment thread core/src/main/java/feast/core/grpc/TrainingServiceImpl.java
@woop
Copy link
Copy Markdown
Member

woop commented Jan 12, 2019 •
edited
Loading

One small thing, I think Datastore should be call FeatureStore, but I'm happy if we refactor that in a new issue.

Calling it FeatureStore might be confusing because we are calling the complete system a "Feature Store".

@zhilingc
Copy link
Copy Markdown
Collaborator Author

zhilingc commented Jan 14, 2019

@woop changes made as requested. Could you review again?

@feast-ci-bot
Copy link
Copy Markdown
Collaborator

feast-ci-bot commented Jan 16, 2019

/lgtm

@feast-ci-bot feast-ci-bot self-assigned this Jan 16, 2019
@tims
Copy link
Copy Markdown
Contributor

tims commented Jan 17, 2019

/approve

@feast-ci-bot
Copy link
Copy Markdown
Collaborator

feast-ci-bot commented Jan 17, 2019

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: tims

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@feast-ci-bot feast-ci-bot merged commit 06cb1ec into feast-dev:master Jan 17, 2019
Yanson pushed a commit to Yanson/feast that referenced this pull request Jul 29, 2020
@ben-voz
KE-805: Add validation logic to spark. (feast-dev#47)
86e8a6b 
KE-805: Add feature validation logic for ingestion with spark.
lokeshrangineni pushed a commit to lokeshrangineni/feast that referenced this pull request Jul 19, 2024
@tmihalac
Merge pull request feast-dev#47 from tmihalac/add-missing-dependecy-p…
4e0357b 
…ermission-cli

Add missing dependecy permission cli
jyejare pushed a commit to jyejare/feast that referenced this pull request Dec 17, 2025
@ntkathole
Merge pull request feast-dev#47 from opendatahub-io/appstudio-feast-k…
d5895be 
…onflux-replicator-v0.55.0-1760495890

Update Tekton files to version v0.55.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@woop woop woop approved these changes

Assignees

@tims tims

@pradithya pradithya

@woop woop

@feast-ci-bot feast-ci-bot

Labels

approved lgtm size/XXL

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@zhilingc @tims @woop @feast-ci-bot @pradithya