Skip to content

chore(packaging): clean project metadata and duplicate dependencies - #629

Merged
DhavalRepo18 merged 1 commit into
IBM:mainfrom
yousseffaouzi:chore/clean-project-metadata
Oct 13, 2026
Merged

DhavalRepo18 merged 1 commit into
IBM:mainfrom
yousseffaouzi:chore/clean-project-metadata

Conversation

@yousseffaouzi

Copy link
Copy Markdown
Contributor

Summary

This PR cleans up the root pyproject.toml by correcting project metadata and removing redundant dependency declarations.

The changes are intentionally small and focused, improving packaging maintainability without modifying application behavior, benchmark logic, or dependency resolution.

Changes Made

1. Update project description

  • Replaced the placeholder Add your description here with:
    MCP tool servers and AI agent runners for industrial asset operations benchmarking

  • Provides a more accurate description of the package and its purpose.

2. Remove duplicate SciPy dependency

  • Removed the redundant scipy>=1.10.0 declaration.

  • Retained the existing requirement without changing its version constraint.

3. Consolidate sktime dependencies

  • Removed the weaker sktime>=0.30 declaration.

  • Retained sktime>=1.0.1, which was already the effective minimum requirement.

  • Preserved the explanatory comment regarding its use in run_recipe and the model catalog.

Impact on Benchmarking

  • No changes to benchmark scoring, evaluation logic, scenarios, or agent behavior.

  • No changes to effective dependency requirements.

  • No changes to the resolved dependency versions.

  • No changes to uv.lock.

Scope: One file modified (pyproject.toml), with 2 insertions and 4 deletions.

Testing & Validation

Validation | Result -- | -- uv lock --check | Passed uv lock | Passed — 284 packages resolved uv sync --frozen | Passed uv run --frozen pytest src/llm/tests src/servers/utilities/tests -q | Passed — 33 tests git diff --check | Passed TSFM service-free test selection | Incomplete — runner time limit exceeded uv.lock comparison | No changes

Additional testing notes:

  • The TSFM test suite was attempted but exceeded the available execution time. A complete TSFM test pass is therefore not claimed.

  • The full non-integration suite was not rerun to completion. A previous run was interrupted with exit code 137 in the constrained environment.

  • Dependency resolution completed successfully without regenerating or modifying the lockfile.

Related Work

Existing PR #331 separately addresses adding Ruff to development dependencies. This contribution intentionally avoids overlapping with that work.

Checklist

  • Changes are minimal and focused.

  • Existing dependency constraints are preserved.

  • Lockfile consistency verified.

  • Focused unit tests passed.

  • No changes to benchmark behavior or scoring.

  • No unrelated dependency upgrades.

  • Commit includes DCO sign-off.

Commit

a88959d5b558c63f00fef3e3a773af021e4f10e9

Branch: chore/clean-project-metadata

Signed-off-by: Youssef Faouzi <46967695+yousseffaouzi@users.noreply.github.com>
yousseffaouzi added a commit to yousseffaouzi/AssetOpsBench that referenced this pull request Oct 12, 2026
Signed-off-by: Youssef Faouzi <46967695+yousseffaouzi@users.noreply.github.com>
@DhavalRepo18
DhavalRepo18 self-requested a review October 12, 2026 21:52
@DhavalRepo18
DhavalRepo18 merged commit e844f64 into IBM:main Oct 13, 2026
3 checks passed
yousseffaouzi added a commit to yousseffaouzi/AssetOpsBench that referenced this pull request Oct 13, 2026
Signed-off-by: Youssef Faouzi <46967695+yousseffaouzi@users.noreply.github.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.

2 participants