Skip to content

docs: fix typos in arXiv extractor and Slicer comments - #2533

Open
Avicennasis wants to merge 1 commit into
NVIDIA-NeMo:mainfrom
TypoScan3000:docs/fix-typos
Open

Avicennasis wants to merge 1 commit into
NVIDIA-NeMo:mainfrom
TypoScan3000:docs/fix-typos

Conversation

@Avicennasis

Copy link
Copy Markdown

Description

Comment and docstring typo fixes:

  • nemo_curator/stages/text/download/arxiv/extract.py: occurence → occurrence (2 comments); remove all in comments within a line → remove all inline comments within a line
  • nemo_curator/stages/text/modifiers/string/slicer.py: occurence → occurrence (2 places in the Slicer docstring)

The commit is GPG-signed and signed off (-sS) as CONTRIBUTING.md asks.

Disclosure: these issues were flagged by an automated typo scanner (TypoScan3000); each one was checked against the current file on the default branch, and the edits were prepared with AI assistance.

No functional changes — comments/docstrings only, so no tests or coverage changes.

Signed-off-by: Léon Avic Simmons <Avicennasis@gmail.com>
@Avicennasis
Avicennasis requested a review from a team as a code owner October 25, 2026 10:02
@Avicennasis
Avicennasis requested review from VibhuJawa and removed request for a team October 25, 2026 10:02
@copy-pr-bot

copy-pr-bot Bot commented Oct 25, 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.

@greptile-apps

greptile-apps Bot commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] This PR appears safe to merge.

Summary

Fixes spelling and wording in comments in ArxivExtractor and argument descriptions in Slicer.

  • The extractor and slicer docs use corrected wording.

Reviews (1) · Last reviewed commit: "docs: fix typos in arXiv extractor and S..." · Reviewed by Greptile

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant