Skip to content

Conversation

@ChrisRackauckas-Claude
Copy link

Summary

  • Removes two completely unused test dependencies: MonteCarloMeasurements and BenchmarkTools
  • Reduces test dependency footprint without affecting any functionality

Analysis Performed

Conducted comprehensive analysis of all test files in the repository to identify unused dependencies:

Dependencies Removed:

  1. BenchmarkTools - No usage found anywhere in tests

    • No @benchmark, @btime, or BenchmarkTools imports in any test files
    • Removed from both [extras] and test targets
  2. MonteCarloMeasurements - No usage found in tests

    • Remains as weak dependency for the extension DiffEqBaseMonteCarloMeasurementsExt
    • Removed only from [extras] and test targets (not from [weakdeps])

Dependencies That Remain:

All other test dependencies are actively used across multiple test files:

  • Aqua: Used in aqua.jl
  • ForwardDiff: Used in 10+ test files for AD testing
  • StaticArrays: Used in 5+ test files
  • Distributions: Used in high_level_solve.jl
  • LabelledArrays: Used in downstream/labelledarrays.jl
  • Measurements: Used in downstream/unwrapping.jl
  • Distributed: Used in downstream/distributed_ensemble.jl
  • And many others...

Test Plan

  • Verified package builds successfully after dependency removal
  • Confirmed no imports or usage of removed packages in any test files
  • Ensured MonteCarloMeasurements extension remains available as weak dependency

Impact

  • Zero functional impact: Removed packages were never used in tests
  • Reduced dependency footprint: Two fewer packages to install for testing
  • Cleaner Project.toml: Removes unused entries that could cause confusion

🤖 Generated with Claude Code

…Tools

After comprehensive analysis of all test files, these packages are never
imported or used in any tests:

- MonteCarloMeasurements: Has weak dependency/extension but no test usage
- BenchmarkTools: No @benchmark, @Btime, or import statements found

Removed from [extras] and test targets. MonteCarloMeasurements remains as
weak dependency for the extension, but is no longer a test dependency.

This reduces the test dependency footprint without affecting functionality.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@ChrisRackauckas ChrisRackauckas merged commit 2827e99 into SciML:master Aug 14, 2025
40 of 48 checks passed
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