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

fix feast repo name in building-and-deploying-multi-stage-RecSys examples #1020

Merged
merged 9 commits into from
Jun 21, 2023

Conversation

rnyak
Copy link
Contributor

@rnyak rnyak commented Jun 20, 2023

We get this error AttributeError: module 'feast' has no attribute 'FeatureStore' from this line feature_store = feast.FeatureStore(feast_repo_path) the reason for that is because the name of the subdirectory is feast . And if the working directory is the directory where the feature store was created, then, import feast imports that directory, and not feast library the package.

this PR fixes the feature store repo name, and also removes the version of feast that we recommend users to install.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@rnyak rnyak added chore Infrastructure update examples Adding new examples labels Jun 20, 2023
@github-actions
Copy link

Documentation preview

https://nvidia-merlin.github.io/Merlin/review/pr-1020

Copy link
Contributor

@nv-alaiacano nv-alaiacano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm once tests pass.

@karlhigley karlhigley dismissed jperez999’s stale review June 21, 2023 14:22

Changes have been made, trying to merge

@karlhigley karlhigley merged commit 30c52cc into main Jun 21, 2023
5 checks passed
@rnyak rnyak deleted the fix_feast_folder_name branch June 21, 2023 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Infrastructure update examples Adding new examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants