Skip to content

Conversation

@SoundsSerious
Copy link
Contributor

This pull request refactors dependency management in pyproject.toml and cleans up unused imports in the codebase. The main changes involve moving several dependencies out of the main install requirements and organizing them into optional dependency groups, which improves maintainability and installation flexibility. Additionally, unused imports are removed from two Python files.

Dependency Management Improvements:

  • Moved multiple packages (such as wheel, jinja2, sphinx, recommonmark, sphinx-rtd-theme, control, fluids, CoolProp, sectionproperties, PyNiteFEA, and expiringdict) from the main dependencies list in pyproject.toml to new optional dependency groups: dev, eng, and google. This streamlines the default installation and makes it easier to install only what's needed for development, engineering, or Google integration. [1] [2] [3]

Code Cleanup:

  • Removed unused imports (expiringdict, attr, attrs) from engforge/dynamics.py and engforge/problem_context.py, reducing clutter and potential confusion. [1] [2]

General Dependency Updates:

  • Removed virtualenv and wheel from the main dependencies, now only included in the dev optional group. [1] [2]

These changes make the project easier to maintain and customize for different use cases.

@SoundsSerious SoundsSerious merged commit 18c126e into main Sep 11, 2025
5 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.

3 participants