Skip to content

Update LangChain component solution notebook #591

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
merged 2 commits into from
Apr 16, 2025
Merged

Update LangChain component solution notebook #591

merged 2 commits into from
Apr 16, 2025
+190 −621

Conversation

takumiohym
Copy link
Collaborator

@takumiohym takumiohym commented Apr 11, 2025 •
edited
Loading

  • Removed toc to align with other notebooks (toc is autogenerated based on Markdown anyway).
  • removed RAG related chunks from Retrieval section (but kept data loaders) and renamed Retrieval to Data Loaders
  • Removed RAG chain from Chain section. Instead Added a sub-chain routing.

The lab notebook will be updated after review.

Merge #583 before before this PR.

@review-notebook-app Review Notebook App
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@takumiohym takumiohym self-assigned this Apr 11, 2025
sanjanalreddy
sanjanalreddy approved these changes Apr 16, 2025
View reviewed changes
Copy link
Collaborator

@sanjanalreddy sanjanalreddy left a comment

Choose a reason for hiding this comment

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

Good to merge, one the navigation menu is added back.

@takumiohym takumiohym changed the base branch from update_langchain_gemini to master April 16, 2025 18:55
@takumiohym takumiohym marked this pull request as ready for review April 16, 2025 18:57
@takumiohym takumiohym merged commit c004662 into master Apr 16, 2025
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sanjanalreddy sanjanalreddy

@kylesteckler kylesteckler

Assignees

@takumiohym takumiohym

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@takumiohym @sanjanalreddy