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

feat: Adding support for Native Python feature transformations for On Demand Feature Views #4045

Merged
merged 16 commits into from
Mar 28, 2024
Merged

feat: Adding support for Native Python feature transformations for On Demand Feature Views #4045

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
da3eba3
uploading current progress, not in a good state yet and have a lot of…
franciscojavierarceo Mar 26, 2024
f8fd949
bug fix
franciscojavierarceo Mar 26, 2024
bbc55de
uploading current progress...calling it a night
franciscojavierarceo Mar 26, 2024
0ac27be
uploading current progress
franciscojavierarceo Mar 27, 2024
b028f8f
updated types and refactored to adjust
franciscojavierarceo Mar 27, 2024
efe16d5
fixed types, substraits, and added tests
franciscojavierarceo Mar 27, 2024
74b77e6
fixed the casting issue
franciscojavierarceo Mar 27, 2024
16850e9
updated mode check
franciscojavierarceo Mar 27, 2024
aa90fc1
fixed lint and udf example...should be singleton
franciscojavierarceo Mar 28, 2024
26c1170
removed print statement
franciscojavierarceo Mar 28, 2024
231e2db
updated to fix integration test
franciscojavierarceo Mar 28, 2024
8fedca7
fixed unit test
franciscojavierarceo Mar 28, 2024
5eb583d
removed import
franciscojavierarceo Mar 28, 2024
b5b71e2
linter
franciscojavierarceo Mar 28, 2024
75a3023
removed print statement and OnlineResponseRow
franciscojavierarceo Mar 28, 2024
c5ecce6
fixed type validation
franciscojavierarceo Mar 28, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
bug fix 
Signed-off-by: Francisco Javier Arceo <franciscojavierarceo@users.noreply.github.com>
  • Loading branch information
@franciscojavierarceo
franciscojavierarceo committed Mar 26, 2024
commit f8fd94915556bd1cf6bd51987bf56e13c7337ed3
2 changes: 0 additions & 2 deletions sdk/python/feast/feature_store.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2134,8 +2134,6 @@ def _augment_response_with_on_demand_transforms(
feature_vector = transformed_features[feature]
proto_values.append(
python_values_to_proto_values(feature_vector, ValueType.UNKNOWN)
python_values_to_proto_values(
transformed_features_df[feature].values, ValueType.UNKNOWN
)

odfv_result_names |= set(selected_subset)
Expand Down
Loading