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

schema has duplicate fields #3459

Closed
hao-affirm opened this issue Jan 20, 2023 · 2 comments · Fixed by #3460
Closed

schema has duplicate fields #3459

hao-affirm opened this issue Jan 20, 2023 · 2 comments · Fixed by #3460

Comments

@hao-affirm
Copy link
Contributor

hao-affirm commented Jan 20, 2023

Expected Behavior

feature_view.schema has no duplicate fields

Current Behavior

feature_view.schema has no duplicate fields

Steps to reproduce

Specifications

  • Version:
  • Platform:
  • Subsystem:

Possible Solution

in feature_view:

    def schema(self) -> List[Field]:
        return list(set(self.entity_columns + self.features))
@felixwang9817
Copy link
Collaborator

felixwang9817 commented Jan 20, 2023

@hao-affirm would you mind sharing the feature view definition that led to duplicate fields shared between self.entity_columns and self.features?

I'm slightly surprised by that, since I thought our inference process would place each Field in either self.entity_columns or self.features, but not both

@hao-affirm
Copy link
Contributor Author

def schema(self) -> List[Field]:

and
for field in schema:
i think both entity_column and features will append the field, if the field is defined in both entities and schema

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

Successfully merging a pull request may close this issue.

2 participants