Skip to content
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

Docs: Revamp Extraction Use Case #18588

Merged
merged 31 commits into from Mar 6, 2024
Merged

Docs: Revamp Extraction Use Case #18588

merged 31 commits into from Mar 6, 2024

Conversation

eyurtsev
Copy link
Collaborator

@eyurtsev eyurtsev commented Mar 5, 2024

Still a draft wrapping up parsing and then doing a typo pass through

@vercel Vercel
Copy link

vercel bot commented Mar 5, 2024 •
edited

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langchain ✅ Ready (Inspect) Visit Preview 💬 2 unresolved
✅ 14 resolved		 Mar 6, 2024 2:13pm

@eyurtsev eyurtsev marked this pull request as ready for review March 5, 2024 19:11
@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Mar 5, 2024
@eyurtsev eyurtsev marked this pull request as draft March 5, 2024 19:11
@dosubot dosubot bot added Ɑ: parsing Related to output parser module 🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder labels Mar 5, 2024
@eyurtsev eyurtsev marked this pull request as ready for review March 5, 2024 21:06
@dosubot dosubot bot added the 🔌: anthropic Primarily related to Anthropic integrations label Mar 5, 2024
@eyurtsev eyurtsev changed the title Extraction Use Case Docs: Revamp Extraction Use Case Mar 5, 2024
@eyurtsev
Copy link
Collaborator Author

eyurtsev commented Mar 5, 2024

@ccurme no need to review just a FYI

ccurme
ccurme reviewed Mar 5, 2024
View reviewed changes
Copy link
Collaborator

@ccurme ccurme left a comment

Choose a reason for hiding this comment

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

Looks great!

docs/docs/use_cases/extraction/index.ipynb Outdated Show resolved Hide resolved
docs/docs/use_cases/extraction/how_to/examples.ipynb Show resolved Hide resolved
@eyurtsev
Copy link
Collaborator Author

eyurtsev commented Mar 6, 2024

This looks good. Merging and making another PR to make a few minor corrections.

@eyurtsev eyurtsev merged commit a4a6978 into master Mar 6, 2024
10 of 11 checks passed
@eyurtsev eyurtsev deleted the eugene/extraction_use_Case branch March 6, 2024 14:18
@baskaryan baskaryan mentioned this pull request Mar 28, 2024
Closed
bechbd pushed a commit to bechbd/langchain that referenced this pull request Mar 29, 2024
@eyurtsev @hwchase17
Docs: Revamp Extraction Use Case (langchain-ai#18588)
45cfd97 
Revamp the extraction use case documentation

---------

Co-authored-by: Harrison Chase <hw.chase.17@gmail.com>
gkorland pushed a commit to FalkorDB/langchain that referenced this pull request Mar 30, 2024
@eyurtsev @hwchase17 @gkorland
Docs: Revamp Extraction Use Case (langchain-ai#18588)
a18cdef 
Revamp the extraction use case documentation

---------

Co-authored-by: Harrison Chase <hw.chase.17@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@baskaryan baskaryan baskaryan left review comments

@ccurme ccurme ccurme left review comments

@hwchase17 hwchase17 Awaiting requested review from hwchase17

Assignees

@baskaryan baskaryan

Labels
🔌: anthropic Primarily related to Anthropic integrations 🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder Ɑ: parsing Related to output parser module size:XXL This PR changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@eyurtsev @hwchase17 @baskaryan @ccurme