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

[Feathr] Add a example for product recommendation #312

Merged
merged 4 commits into from
Jun 1, 2022
Merged

Conversation

hangfei
Copy link
Collaborator

@hangfei hangfei commented Jun 1, 2022

Update the sample with a product recommendation example.

To make it simpler and easy to understand, we are only rate ONE product for different users.

In future series, we will add rating for arbitrary products.

I will add a few more illustrations in next PR.

@xiaoyongzhu xiaoyongzhu self-requested a review June 1, 2022 18:59
@xiaoyongzhu
Copy link
Member

Looks good to me. Can you open an issue so that someone else can add more illustration for this example?

@hangfei
Copy link
Collaborator Author

hangfei commented Jun 1, 2022

Looks good to me. Can you open an issue so that someone else can add more illustration for this example?

I have a illustration already. It's just easier to do it in separate PR so i don't have to run the tests again.

@xiaoyongzhu xiaoyongzhu merged commit aa5f601 into main Jun 1, 2022
@xiaoyongzhu xiaoyongzhu deleted the product_sample branch June 1, 2022 20:59
bozhonghu pushed a commit that referenced this pull request Jun 2, 2022
* main:
  added init py for import error (#308)
  add contribution text (#153)
  Add 'feathr hocon' command (#317)
  Feathr UI: API spec alignment and ux experience improvement (#303)
  bump version (#315)
  Adding custom data loader handler support. (#309)
  Update product_recommendation_demo.ipynb
  Update product_recommendation_demo.ipynb
  Product sample (#314)
  [Feathr] Add a example for product recommendation (#312)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants