Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 19, 2026

Consolidate Testing Dependencies in pyproject.toml

All Tasks Complete ✅

  • Analyzed repository structure and testing setup
  • Identified scattered testing dependencies across 8 workflow files
  • Created [dev] optional dependency group in pyproject.toml with core testing packages
  • Updated all workflow files to use pip install .[dev] with manual installation of extra packages
  • Removed duplicate dependency installation steps from workflows
  • Fixed duplicate osmnx dependency (kept in advanced group only)
  • Updated tests/README.md with accurate installation instructions
  • Removed [extra] group per maintainer request - packages installed manually
  • Updated workflow comments to accurately reflect what's being installed

Summary of Changes

pyproject.toml:

  • Added [dev] group: pytest>=8.4.0, pytest-xdist>=3.8.0, pytest-rerunfailures>=16.0, pytest-cov, setuptools, jupyter, nbformat
  • Kept osmnx>=2.0.0 in existing [advanced] group (with neatnet, geopandas, shapely)
  • Removed [extra] group - packages (gymnasium, torch, deap, streamlit, requests) are now installed manually in workflow files

Workflow Updates (5 files):

  • test-functions.yml: Uses .[dev] + manually installs osmnx, requests
  • run-examples.yml: Uses .[dev] + manually installs gymnasium, torch, osmnx, deap, streamlit
  • run-notebook-demos-extra.yml: Uses .[dev] + manually installs gymnasium, torch, osmnx, deap
  • measure-coverage.yml: Uses .[dev,advanced] + manually installs osmnx, requests
  • test-functions-python313.yml: Uses .[dev,advanced] + manually installs requests
  • Updated comments from "Install testing dependencies with compatible versions for parallel execution" to "Install optional packages required for these tests" to accurately reflect the new structure

Documentation:

  • Updated tests/README.md to instruct users to manually install extra packages

Benefits

✅ Single source of truth for core testing dependencies in pyproject.toml
✅ Easier maintenance for core test tools
✅ Extra packages installed manually as requested by maintainer
✅ Consistent dependency versions for core test infrastructure
✅ Clear and accurate comments in workflow files

Original prompt

This section details on the original issue you should resolve

<issue_title>Inconsistency in test suite</issue_title>
<issue_description>Currently, there are minor issues with the test scripts in the /tests folder and the test execution workflows under /.github/workflows.
Specifically, package information for testing is scattered across various locations.
This could be properly consolidated by creating a [dev] option in pyproject.toml.

Please resolve this issue and verify that CI can execute all tests successfully.

Task for Copilot.</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Fix inconsistency in test suite and workflows Consolidate scattered testing dependencies into pyproject.toml Jan 19, 2026
Copilot AI requested a review from toruseo January 19, 2026 10:30
@toruseo
Copy link
Owner

toruseo commented Jan 20, 2026

@copilot basically ok, but please undo [extra] option. they should be installed manually

Copy link
Contributor Author

Copilot AI commented Jan 20, 2026

@copilot basically ok, but please undo [extra] option. they should be installed manually

Done! Removed the [extra] group from pyproject.toml. Extra packages (gymnasium, torch, deap, streamlit, requests) are now installed manually in each workflow file that needs them. Commit: 880c586

Copilot AI changed the title Consolidate scattered testing dependencies into pyproject.toml Consolidate core testing dependencies into pyproject.toml [dev] group Jan 20, 2026
@toruseo
Copy link
Owner

toruseo commented Jan 20, 2026

@copilot update the yml comments like "# Install testing dependencies with compatible versions for parallel execution"

Copy link
Contributor Author

Copilot AI commented Jan 20, 2026

@copilot update the yml comments like "# Install testing dependencies with compatible versions for parallel execution"

Updated the comments in all workflow files from "Install testing dependencies with compatible versions for parallel execution" to "Install optional packages required for these tests" to accurately reflect that testing dependencies are now in the [dev] group and these steps only install optional packages. Commit: d94c86e

@toruseo toruseo marked this pull request as ready for review January 20, 2026 02:25
@toruseo toruseo enabled auto-merge January 20, 2026 02:25
@toruseo toruseo merged commit 218e51e into main Jan 20, 2026
15 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.

Inconsistency in test suite

2 participants