Fix: Add missing latex2sympy2_extended to requirements

[shantoislamdev]

Description

Adds missing dependency latex2sympy2_extended to requirements/main.txt.

Problem

The package latex2sympy2_extended is imported by nemo_skills/evaluation/math_grader.py but was not listed in the project requirements. This causes a ModuleNotFoundError when users install the package and attempt to run math evaluation pipelines (e.g., GSM8K, AIME, or any benchmark using the math metric type).

Import chain:

nemo_skills/pipeline/eval.py 
  → nemo_skills/pipeline/summarize_results.py
    → nemo_skills/evaluation/metrics/compute_metrics.py
      → nemo_skills/evaluation/metrics/math_metrics.py
        → nemo_skills/evaluation/math_grader.py
          → from latex2sympy2_extended import NormalizationConfig, normalize_latex

Solution

Add latex2sympy2_extended to requirements/main.txt after language-data to maintain alphabetical ordering.

Testing

After this change, users can successfully:

1. Install the package: pip install -e .
2. Run math evaluations without ModuleNotFoundError

Impact

• Breaking: No
• User-facing: Yes (fixes broken installation for math evaluation)
• Dependencies: Adds one package to installation requirements

Summary by CodeRabbit

• Chores
  • Updated project dependencies to enhance system capabilities.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[greptile-apps]

_{1 file reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[coderabbitai]

📝 Walkthrough

Walkthrough

Added the latex2sympy2_extended dependency to the project requirements without modifying existing entries. This extends the installation set to include math evaluation capabilities via LaTeX to SymPy conversion.

Changes

Cohort / File(s)	Summary
Dependency Addition requirements/main.txt	Added latex2sympy2_extended to project dependencies for extended LaTeX to SymPy conversion support.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: adding a missing dependency (latex2sympy2_extended) to requirements/main.txt to fix a ModuleNotFoundError in math evaluation.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[greptile-apps]

_{1 file reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

_{No files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}

[Kipok]

this should already be installed as a dependency of math-verify. @shantoislamdev could you please share the command you're running that causes this error?

[shantoislamdev]

ah right, makes sense. math-verify already pulls it in. saw it imported in math_grader but not listed explicitly in requirements and I assumed it was a missing top-level dependency, didn't think to check transitive deps. my bad, closing!