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

Returned feature vector when on demand feature views are specified output dependent features #1999

Open
adchia opened this issue Nov 4, 2021 · 4 comments

Comments

@adchia
Copy link
Collaborator

adchia commented Nov 4, 2021

When specifying on demand feature views at retrieval time (e.g. get_X_features), the output feature vectors include e.g. request data or dependent feature vectors, even if users did not specify said features.

Expected Behavior

Non-specified dependent feature values are not returned in output

Current Behavior

Non-specified dependent feature values are in output

Steps to reproduce

Specifications

  • Version:
  • Platform:
  • Subsystem:

Possible Solution

@felixwang9817
Copy link
Collaborator

Resolved by #2004

@adchia adchia reopened this Nov 8, 2021
@adchia
Copy link
Collaborator Author

adchia commented Nov 8, 2021

This isn't resolved yet for get_historical_features

@stale
Copy link

stale bot commented May 25, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label May 25, 2022
@stale stale bot removed the wontfix This will not be worked on label May 26, 2022
@kevjumba kevjumba added Community Contribution Needed We want community to contribute critical labels Aug 3, 2022
@tmihalac
Copy link
Contributor

Hello, I am new to Feast can you please explain to in detail what is needed to fix this for get_historical_features ?

This is what Tornike Gurgenidze answered in Slack :
"Hi, most of the code you might be interested in is in RetrievalJob class (https://github.com/feast-dev/feast/blob/master/sdk/python/feast/infra/offline_stores/offline_store.py#L61), look for to_df and to_arrow methods. When individual feature store implementations create subclasses of RetrievalJob, they normally override _to_df_internal and _to_arrow_internal methods only. to_df then calls _to_df_internal , applies final logic to calculate values for odfvs and returns the final output.

Also, I might be totally wrong about this, but in case of get_historical_features I don't think the bug is that extra unspecified feature values are being returned, but rather that if the user doesn't specify all the features that requested odfv features depend on, RetrievalJob will be unable to calculate feature views and the call will most likely fail.
offline_store.py
class RetrievalJob(ABC):"

I need further explanation about it and also how to reproduce the bug to be able to test the fix.

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

No branches or pull requests

4 participants