Skip to content

Comments

[UNIT] New unit on a Agentic RAG use case (Unit 3)#307

Merged
sergiopaniego merged 41 commits intounit-3-agentic-ragfrom
add-unit3
Mar 27, 2025
Merged

[UNIT] New unit on a Agentic RAG use case (Unit 3)#307
sergiopaniego merged 41 commits intounit-3-agentic-ragfrom
add-unit3

Conversation

@burtenshaw
Copy link
Collaborator

@burtenshaw burtenshaw commented Mar 11, 2025

This PR add Unit 3 on an Agentic RAG use case:

It builds opon these PRs:

Remaining work on this PR

  • review material, code, and copy
  • define student's approach to use case. i.e. should the build the agent in a space
  • add frame works tabs for LlamaIndex and LangGraph

@burtenshaw burtenshaw changed the title [unit] unit 3 on use case [UNIT] New unit on a Agentic RAG use case (Unit 3) Mar 11, 2025
burtenshaw and others added 2 commits March 17, 2025 16:05
Co-authored-by: David Berenstein <david.m.berenstein@gmail.com>
@sergiopaniego
Copy link
Member

I've created a PR with some suggestions 😄

A few points that also need to be addressed:

  • In invitees.mdx, a unit3-invitees dataset is mentioned, but it isn't available yet. Has it already been created?
  • There are mentions of Python files throughout the text (e.g., retriever.py and tools.py). I find this a bit confusing. Perhaps adding a file structure overview would help clarify things.
  • Directly related to the previous point, no notebooks/Colab files are included in the PR.

I can work on any of these. Just leaving the points here to avoid duplicating work 😄

@burtenshaw
Copy link
Collaborator Author

Thanks @sergiopaniego. I merged your PR. Replying in line:

In invitees.mdx, a unit3-invitees dataset is mentioned, but it isn't available yet. Has it already been created?

No, we need to create this dataset.

There are mentions of Python files throughout the text (e.g., retriever.py and tools.py). I find this a bit confusing. Perhaps adding a file structure overview would help clarify things.

The idea was that students use a template repo with empty files in that they need to complete to make the use case work. This could be a space.

Directly related to the previous point, no notebooks/Colab files are included in the PR.

No notebooks here. The idea was that students make an application, but you're right it's not clear from the current material.

@sergiopaniego
Copy link
Member

Thanks for the details @burtenshaw

I've generated #340 for the LlamaIndex tab.

@sergiopaniego sergiopaniego changed the base branch from main to unit-3-agentic-rag March 27, 2025 09:29
@HuggingFaceDocBuilderDev
Copy link
Collaborator

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@sergiopaniego sergiopaniego merged commit 950257a into unit-3-agentic-rag Mar 27, 2025
1 check passed
@sergiopaniego sergiopaniego deleted the add-unit3 branch March 27, 2025 09:59
giacomosansoni pushed a commit to giacomosansoni/agents-course that referenced this pull request May 17, 2025
[UNIT] New unit on a Agentic RAG use case (Unit 3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants