Skip to content

Add a Sandbox for Feathr #966

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

Merged
merged 88 commits into from
Jan 17, 2023
Merged

Conversation

xiaoyongzhu
Copy link
Member

@xiaoyongzhu xiaoyongzhu commented Jan 13, 2023

Goal of this PR is to package everything into a single docker image to:

  • make it easier for users to get started,
  • make it easy to validate feature definitions and new ideas
  • make it easier for Feathr devs to setup environment and develop new things
  • Interactive experience, usually try to run a job takes less than 1 min.

As an end user, they can become productive in less than 5 mins.

Users can try it out by following this doc:
https://xiaoyongzhu.github.io/feathr/quickstart_local_sandbox.html

[x] Currently feature materialization doesn't work in the latest branch, although using Feathr 0.9 it worked in sandbox environment. @Yuqing-cat
[x] There are some potential version conflicts when installing Feathr in Sandbox.
[x] Search functionality in Sandbox doesn't work
[x] Updated README files and instructions for users to get started
[x] Remove login UI for Microsoft account
[x] Remove duplicated code
[x] Make Feathr API auth a bit more configurable
[x] merge the ORM based SQL backend to sql-registry folder

Copy link
Collaborator

@loomlike loomlike left a comment

Choose a reason for hiding this comment

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

This is super cool! I haven't actually run in on my local box yet, but I will try and let you know if I have any issues.
In the meantime, I left some comments as well. Thank you!

@enya-yx enya-yx added the registry test Used to run test cases for registry label Jan 17, 2023
Yuqing-cat
Yuqing-cat previously approved these changes Jan 17, 2023
@xiaoyongzhu xiaoyongzhu merged commit 290ceb3 into feathr-ai:main Jan 17, 2023
@xiaoyongzhu xiaoyongzhu deleted the feathr-sandbox branch January 17, 2023 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
registry test Used to run test cases for registry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants