Skip to content

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