-
Notifications
You must be signed in to change notification settings - Fork 129
Nvingest curator tutorial #584
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.
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
Conversation
Signed-off-by: Rucha Apte <[email protected]>
Signed-off-by: Rucha Apte <[email protected]>
Signed-off-by: Rucha Apte <[email protected]>
Signed-off-by: Rucha Apte <[email protected]>
Signed-off-by: Rucha Apte <[email protected]>
Signed-off-by: Rucha Apte <[email protected]>
Signed-off-by: Rucha Apte <[email protected]>
pre-commit.ci autofix |
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm! Just had one small nitpick
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One more thing
Co-authored-by: ChrisJar <[email protected]> Signed-off-by: Rucha Apte <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very clean code, thank you. Added a bunch of nits.
tutorials/multimodal_dapt_curation/curator/configs/struct_semantic_dedupe_config.yaml
Outdated
Show resolved
Hide resolved
tutorials/multimodal_dapt_curation/curator/configs/text_semantic_dedupe_config.yaml
Outdated
Show resolved
Hide resolved
tutorials/multimodal_dapt_curation/curator/configs/struct_semantic_dedupe_config.yaml
Outdated
Show resolved
Hide resolved
tutorials/multimodal_dapt_curation/curator/configs/text_semantic_dedupe_config.yaml
Outdated
Show resolved
Hide resolved
If needed, could you add |
Co-authored-by: Sarah Yurick <[email protected]> Signed-off-by: Rucha Apte <[email protected]>
Co-authored-by: Sarah Yurick <[email protected]> Signed-off-by: Rucha Apte <[email protected]>
Co-authored-by: Sarah Yurick <[email protected]> Signed-off-by: Rucha Apte <[email protected]>
Co-authored-by: Sarah Yurick <[email protected]> Signed-off-by: Rucha Apte <[email protected]>
Co-authored-by: Sarah Yurick <[email protected]> Signed-off-by: Rucha Apte <[email protected]>
…ntic_dedupe_config.yaml Co-authored-by: Sarah Yurick <[email protected]> Signed-off-by: Rucha Apte <[email protected]>
…ic_dedupe_config.yaml Co-authored-by: Sarah Yurick <[email protected]> Signed-off-by: Rucha Apte <[email protected]>
…ntic_dedupe_config.yaml Co-authored-by: Sarah Yurick <[email protected]> Signed-off-by: Rucha Apte <[email protected]>
…ic_dedupe_config.yaml Co-authored-by: Sarah Yurick <[email protected]> Signed-off-by: Rucha Apte <[email protected]>
Co-authored-by: Sarah Yurick <[email protected]> Signed-off-by: Rucha Apte <[email protected]>
Co-authored-by: Sarah Yurick <[email protected]> Signed-off-by: Rucha Apte <[email protected]>
Co-authored-by: Sarah Yurick <[email protected]> Signed-off-by: Rucha Apte <[email protected]>
Co-authored-by: Sarah Yurick <[email protected]> Signed-off-by: Rucha Apte <[email protected]>
Signed-off-by: Rucha Apte <[email protected]>
Signed-off-by: Rucha Apte <[email protected]>
Signed-off-by: Rucha Apte <[email protected]>
pre-commit.ci autofix |
for more information, see https://pre-commit.ci
tutorials/multimodal_dapt_curation/curator/configs/struct_semantic_dedupe_config.yaml
Outdated
Show resolved
Hide resolved
tutorials/multimodal_dapt_curation/curator/configs/text_semantic_dedupe_config.yaml
Outdated
Show resolved
Hide resolved
Signed-off-by: Rucha Apte <[email protected]>
Signed-off-by: Rucha Apte <[email protected]>
pre-commit.ci autofix |
for more information, see https://pre-commit.ci
Signed-off-by: Rucha Apte <[email protected]>
…-apte/NeMo-Curator into nvingest_curator_tutorial
pre-commit.ci autofix |
for more information, see https://pre-commit.ci
Signed-off-by: Rucha Apte <[email protected]>
Signed-off-by: Rucha Apte <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
[heart] Rucha Apte reacted to your message:
…________________________________
From: Sarah Yurick ***@***.***>
Sent: Tuesday, April 29, 2025 9:32:52 PM
To: NVIDIA/NeMo-Curator ***@***.***>
Cc: Rucha Apte ***@***.***>; Author ***@***.***>
Subject: Re: [NVIDIA/NeMo-Curator] Nvingest curator tutorial (PR #584)
@sarahyurick approved this pull request.
LGTM, thank you!
—
Reply to this email directly, view it on GitHub<#584 (review)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/BMJYJVTFBTADB3NYTEU436L237VYJAVCNFSM6AAAAABYZ7F5SSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDQMBVGEZTGMJWGA>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Closing in favor of #699. Still waiting to see if the team can relax some of the requirements around signed GitHub commits... |
Description
This tutorial is divided into two parts:
Part 1: Multimodal Extraction - guide to extracting various modalities (text, images, tables, etc.) from PDFs using NVIDIA's multimodal extraction (nv-ingest) framework.
Part 2: Data Curation for Domain-Adaptive Pre-Training (DAPT) - covers best practices for data curation in DAPT. This stage processes extracted text, tables, charts, and images using the curation pipeline.
Usage
Follow the README under ingest first and then curator folder for installation of pre-reqs
Checklist