Add a simple notebook for llama stack #244

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.

Sign up for GitHub

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

Jump to bottom

Merged

Amnah199 merged 6 commits into main from add-llama-stack-cookbook

Jul 21, 2025

Contributor

Amnah199 commented Jul 17, 2025

No description provided.


          Add a simple notebook for llama stack

8c124a3

Amnah199 requested a review from a team as a code owner

July 17, 2025 16:22

review-notebook-app bot commented Jul 17, 2025

Check out this pull request on

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

davidsbatista reviewed

View reviewed changes

notebooks/llama_stack_with_tools.ipynb Outdated Show resolved Hide resolved


          PR comments

3a139af

davidsbatista reviewed

View reviewed changes

notebooks/llama_stack_with_tools.ipynb Outdated Show resolved Hide resolved


          PR comments

88552b5

Amnah199 requested a review from davidsbatista

July 18, 2025 12:00

Contributor

davidsbatista commented Jul 18, 2025

sorry, one last comment, I think you need to remove the ## Installation from the cell, the %%bash needs to be the only thing at the header - you can merge after

davidsbatista approved these changes

View reviewed changes

Contributor

davidsbatista left a comment

just one small comment, merge it after

Amnah199 added 2 commits

July 21, 2025 11:24


          Update the notebook

43586bf


          rename the file

7851c96

bilgeyucel reviewed

View reviewed changes

Contributor

bilgeyucel left a comment

Thanks @Amnah199! I left my comments

index.toml Outdated Show resolved Hide resolved


          Update index.toml

85ba5b2

Co-authored-by: Bilge Yücel <[email protected]>

Amnah199 merged commit f974980 into main

1 check passed

Amnah199 deleted the add-llama-stack-cookbook branch

July 21, 2025 09:46

bilgeyucel reviewed

View reviewed changes

notebooks/llama_stack_with_agent.ipynb

		@@ -0,0 +1,225 @@
		{

Contributor

bilgeyucel Jul 21, 2025 •

edited

Loading

Can you link to the agent documentation page here?

Reply via ReviewNB

notebooks/llama_stack_with_agent.ipynb

		@@ -0,0 +1,225 @@
		{

Contributor

bilgeyucel Jul 21, 2025 •

edited

Loading

Link to the Tool documentation page

Reply via ReviewNB

notebooks/llama_stack_with_agent.ipynb

		@@ -0,0 +1,225 @@
		{

Contributor

bilgeyucel Jul 21, 2025 •

edited

Loading

Let's add more info here to explain the relation between two:

"Now, let's create a LlamaStackChatGenerator and pass it to the Agent. The Agent component will use the model running with LlamaStackChatGenerator to reason and make decisions."

Reply via ReviewNB

notebooks/llama_stack_with_agent.ipynb

		@@ -0,0 +1,225 @@
		{

Contributor

bilgeyucel Jul 21, 2025 •

edited

Loading

"Now, when we ask questions, the Agent will utilize both the provided tool and the LlamaStackChatGenerator to generate answers. We enable the streaming in Agent through streaming_callback, so you can observe the tool calls and results in real time."

Reply via ReviewNB

notebooks/llama_stack_with_agent.ipynb

		@@ -0,0 +1,225 @@
		{

Contributor

bilgeyucel Jul 21, 2025 •

edited

Loading

The markdown here needs to get fixed:

"[Llama Stack docs](https://llama-stack.readthedocs.io/en/latest/providers/inference/index.html)."

Reply via ReviewNB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet