Skip to content

docs: Adding example of RHOAI feast quick start guide#4564

Merged
franciscojavierarceo merged 4 commits into
feast-dev:masterfrom
lokeshrangineni:feature/rhoai-feast-example
Oct 10, 2024
Merged

docs: Adding example of RHOAI feast quick start guide#4564
franciscojavierarceo merged 4 commits into
feast-dev:masterfrom
lokeshrangineni:feature/rhoai-feast-example

Conversation

@lokeshrangineni

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:

Adding example of RHOAI feast quick start guide

Which issue(s) this PR fixes:

Misc

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>
@lokeshrangineni lokeshrangineni changed the title Adding example of RHOAI feast quick start guide docs: Adding example of RHOAI feast quick start guide Sep 24, 2024
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb Outdated
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb Outdated
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb Outdated
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb Outdated
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb Outdated
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb Outdated
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb
@dmartinol

Copy link
Copy Markdown
Contributor

I miss the RHOAI part of the example. I think we can run the same notebook as-is even from a local repo, right?

@lokeshrangineni

Copy link
Copy Markdown
Contributor Author

I miss the RHOAI part of the example. I think we can run the same notebook as-is even from a local repo, right?

Yes, this notebook can be run on it is own on the local. Initial expectation is only to provide basics of feast without pipelines.

Adding the missed link in the index page of root examples directory.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

@dmartinol dmartinol left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm, but still the RHOAI part is pending. Please clarify in the PR description if you plan to add further, more specific content.

@lokeshrangineni

Copy link
Copy Markdown
Contributor Author

lgtm, but still the RHOAI part is pending. Please clarify in the PR description if you plan to add further, more specific content.

I am planning to add another example with RHOAI pipelines and feast. The initial scope is only focused on basics of feast.

@dandawg dandawg left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm glad to see that we have a notebook example. I feel like some of the "why" is missing, and I think this may stem from the data itself. The data is auto-generated and has little meaning.

Comment thread examples/rhoai-quickstart/README.md
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb Outdated
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb Outdated
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb Outdated
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb Outdated
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb Outdated
Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb Outdated
"import subprocess\n",
"\n",
"# Run feast serve in the background\n",
"feast_remote_registry_server_process = subprocess.Popen([\"feast\", \"serve_registry\"])\n",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This cell didn't run for me. I got a "ModuleNotFound" error for grpc. Once I ran pip install grpcio, then it worked. This probably needs to be added to the top of the notebook.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I am a bit surprised why you had to install it separately. I assumed it will be part of feast dependencies but anyway added this dependency now. PTAL.

Comment thread examples/rhoai-quickstart/feast-demo-quickstart.ipynb
@franciscojavierarceo

Copy link
Copy Markdown
Member

is this PR ready for review?

Added more details about RHOAI platform and reference to the documentation.
Incorporated other minor comments.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>
@lokeshrangineni

Copy link
Copy Markdown
Contributor Author

@dandawg - Please take a look at the PR again, I have incorporated your code review comments.

"tags": []
},
"source": [
"### Starting feast online sever"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
"### Starting feast online sever"
"### Starting feast online feature server"

@franciscojavierarceo franciscojavierarceo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@franciscojavierarceo franciscojavierarceo merged commit cd87562 into feast-dev:master Oct 10, 2024
@lokeshrangineni lokeshrangineni deleted the feature/rhoai-feast-example branch October 23, 2024 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants