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

Add feature embedding example #881

Merged
merged 4 commits into from
Dec 2, 2022

Conversation

loomlike
Copy link
Collaborator

@loomlike loomlike commented Nov 28, 2022

Signed-off-by: Jun Ki Min 42475935+loomlike@users.noreply.github.com

Description

This PR includes:

  • Feature embedding example
  • Increase databricks sample notebook's timeout argument
  • Add samples description to README

How was this PR tested?

pytest under test/samples
Feature registration part is bypassed.
Once we support proper cleanup features, I can add the registration tests as well (#880).

Does this PR introduce any user-facing changes?

  • No. You can skip the rest of this section.
  • Yes. Make sure to clarify your proposed changes.

Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com>
xiaoyongzhu
xiaoyongzhu previously approved these changes Nov 28, 2022
@loomlike
Copy link
Collaborator Author

Hi team, I added "Samples" section to docs/README.md describing about sample notebooks.
Please feel free to share any comments around samples' descriptions since I may miss some important purpose of those examples.

Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com>
@loomlike loomlike added the safe to test Tag to execute build pipeline for a PR from forked repo label Nov 29, 2022
Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com>
Copy link
Collaborator

@blrchen blrchen left a comment

Choose a reason for hiding this comment

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

Thanks Jun, this is a nice PR!

@xiaoyongzhu xiaoyongzhu merged commit 0955d15 into feathr-ai:main Dec 2, 2022
jaymo001 pushed a commit that referenced this pull request Dec 7, 2022
* Add feature embedding example. Update README

Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com>

* Add feature embedding notebook test

Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com>

* increase notebook's spark job timeout

Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com>

Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe to test Tag to execute build pipeline for a PR from forked repo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants