Skip to content

docs: update docs and release notes for fused iterate-extract (PR #1458) - #1684

Merged
lbliii merged 14 commits into
26.04-stagingfrom
lbliii/classifier-docs-1458
Apr 6, 2026
Merged

docs: update docs and release notes for fused iterate-extract (PR #1458)#1684
lbliii merged 14 commits into
26.04-stagingfrom
lbliii/classifier-docs-1458

Conversation

@lbliii

@lbliii lbliii commented Mar 30, 2026

Copy link
Copy Markdown
Contributor

Description

Updates v26.04 fern documentation and release notes to reflect the fused document iterate and extract stages from #1458. Replaces the 26.02 fern release notes with a 26.04 skeleton including the fused stages entry and breaking changes. Updates the data acquisition concepts page from four-stage to three-stage architecture, updates the custom data loading guide from 4-step to 3-step, and adds a 26.04 section to the non-fern release notes with matching content.

Usage

from nemo_curator.stages.text.download.base.iterator import DocumentIterateExtractStage

# Before (26.02): separate iterate and extract stages
# iterate_stage = DocumentIterateStage(iterator=my_iterator)
# extract_stage = DocumentExtractStage(extractor=my_extractor)

# After (26.04): fused into a single stage
iterate_extract_stage = DocumentIterateExtractStage(
    iterator=my_iterator,
    extractor=my_extractor,  # optional
    record_limit=100,
    add_filename_column=True,
)

Checklist

  • I am familiar with the Contributing Guide.
  • New or Existing tests cover these changes.
  • The documentation is up to date with these changes.

@lbliii
lbliii requested review from a team as code owners March 30, 2026 19:05
@lbliii
lbliii requested review from VibhuJawa and removed request for a team March 30, 2026 19:05

@jgerh jgerh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Completed tech pubs review and provided a few copyedits

Comment thread docs/about/release-notes/index.md
Comment thread docs/about/release-notes/index.md
Comment thread docs/about/release-notes/index.md
Comment thread docs/about/release-notes/index.md
Comment thread docs/about/release-notes/index.md
Comment thread fern/versions/v26.04/pages/curate-text/load-data/custom.mdx Outdated
Comment thread fern/versions/v26.04/pages/curate-text/load-data/custom.mdx Outdated
Comment thread fern/versions/v26.04/pages/curate-text/load-data/custom.mdx Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Delete horizontal divider in the rendered output

Comment thread fern/versions/v26.04/pages/curate-text/load-data/custom.mdx Outdated
lbliii and others added 10 commits March 31, 2026 09:40
…(PR #1458)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Lawrence Lane <llane@nvidia.com>
Revert 26.02 release notes line to avoid referencing 26.04 class name
in wrong version section. Update custom.mdx wording to reflect that the
fused step maps to multiple abstract base classes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Lawrence Lane <llane@nvidia.com>
Apply tech pubs copyedits: add periods to list items, fix link spacing,
backtick code references, correct product names (FastText, GLiNER),
replace OOM'd with "ran out of memory", rewrite component descriptions
for clarity, convert <Note> to :::{note} directive, remove horizontal
dividers, and align fern/non-fern breaking changes (add Three-Stage
Pipeline entry to non-fern).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Lawrence Lane <llane@nvidia.com>
Release notes live in fern only; remove docs/ changes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Lawrence Lane <llane@nvidia.com>
Co-authored-by: jgerh <163925524+jgerh@users.noreply.github.com>
Signed-off-by: Lawrence Lane <llane@nvidia.com>
Co-authored-by: jgerh <163925524+jgerh@users.noreply.github.com>
Signed-off-by: Lawrence Lane <llane@nvidia.com>
Co-authored-by: jgerh <163925524+jgerh@users.noreply.github.com>
Signed-off-by: Lawrence Lane <llane@nvidia.com>
Co-authored-by: jgerh <163925524+jgerh@users.noreply.github.com>
Signed-off-by: Lawrence Lane <llane@nvidia.com>
Co-authored-by: jgerh <163925524+jgerh@users.noreply.github.com>
Signed-off-by: Lawrence Lane <llane@nvidia.com>
Co-authored-by: jgerh <163925524+jgerh@users.noreply.github.com>
Signed-off-by: Lawrence Lane <llane@nvidia.com>
@lbliii

lbliii commented Apr 1, 2026

Copy link
Copy Markdown
Contributor Author

@claude review

@lbliii

lbliii commented Apr 1, 2026

Copy link
Copy Markdown
Contributor Author

/claude review

Comment on lines +248 to +250
:::{note}
Data acquisition includes basic content-level deduplication during extraction (such as removing duplicate HTML content within individual web pages). This is separate from the main deduplication pipeline stages (exact, fuzzy, and semantic deduplication) that operate on the full dataset after acquisition.
</Note>
:::

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Bug: The rest of the v26.04 fern docs consistently use <Note>/</Note> syntax (e.g., installation.mdx, dedup.mdx, index.mdx). This :::{note} / ::: syntax is inconsistent and may not render correctly in fern.

Suggested change
:::{note}
Data acquisition includes basic content-level deduplication during extraction (such as removing duplicate HTML content within individual web pages). This is separate from the main deduplication pipeline stages (exact, fuzzy, and semantic deduplication) that operate on the full dataset after acquisition.
</Note>
:::
<Note>
Data acquisition includes basic content-level deduplication during extraction (such as removing duplicate HTML content within individual web pages). This is separate from the main deduplication pipeline stages (exact, fuzzy, and semantic deduplication) that operate on the full dataset after acquisition.
</Note>

Comment thread docs/about/release-notes/index.md Outdated
- **InternVideo2 Removed**: Video pipelines must use alternative embedding models (Cosmos-Embed1)
- **ID Field Standardization**: Custom deduplication workflows may need updates to use standardized ID field names
- **InternVideo2 Removed**: Video pipelines must use alternative embedding models (Cosmos-Embed1).
- **ID Field Standardization**: Custom deduplication workflows may need updates to use standardized ID field names.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

fyi this one was for 26.04, not 26.02 (#1390). Can't tell which one it's supposed to be associated with here.

Remove ID Field Standardization entries from non-fern 26.02 release
notes since PR #1390 is a 26.04 feature. Remove fused iterate-extract
entry from same section (26.04 via PR #1458). Remove remaining
horizontal divider in custom.mdx.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Lawrence Lane <llane@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Apr 6, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

lbliii and others added 2 commits April 6, 2026 11:26
Restore docs/about/release-notes/index.md to staging state; this PR
should not modify the non-fern release notes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Lawrence Lane <llane@nvidia.com>
Keep both fused iterate-extract stages (PR #1458) and new staging
content (inference server, workflow results, CVE fixes, etc.) in the
fern release notes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Lawrence Lane <llane@nvidia.com>
Keep fused iterate-extract entry (PR #1458) and add pipeline stage
metrics entry (PR #1385) from upstream.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

3 participants