Skip to content

Comments

Remove usage of pkg_resources which has been deprecated#1732

Merged
blnicho merged 1 commit intoIDAES:mainfrom
blnicho:remove-pkg-resources
Feb 19, 2026
Merged

Remove usage of pkg_resources which has been deprecated#1732
blnicho merged 1 commit intoIDAES:mainfrom
blnicho:remove-pkg-resources

Conversation

@blnicho
Copy link
Member

@blnicho blnicho commented Feb 12, 2026

Summary/Motivation:

pkg_resources has been removed from setuptools. Most common uses of pkg_resources have been superseded by importlib.metadata. This PR replaces uses of pkg_resources with functionality from importlib.metadata.

Changes proposed in this PR:

  • Replace deprecated uses of pkg_resources

Legal Acknowledgement

By contributing to this software project, I agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the license terms described in the LICENSE.txt file at the top level of this directory.
  2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

@blnicho blnicho added the CI:run-integration triggers_workflow: Integration label Feb 12, 2026
@idaes-build idaes-build removed the CI:run-integration triggers_workflow: Integration label Feb 12, 2026
@blnicho
Copy link
Member Author

blnicho commented Feb 12, 2026

Integration tests are failing due to an example file that is using pkg_resources. I'll open a PR into the examples repo to fix the notebooks and link it.

@codecov
Copy link

codecov bot commented Feb 12, 2026

Codecov Report

❌ Patch coverage is 77.77778% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.67%. Comparing base (da90c69) to head (a19440e).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
idaes/core/util/env_info.py 77.77% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1732   +/-   ##
=======================================
  Coverage   73.67%   73.67%           
=======================================
  Files         398      398           
  Lines       65101    65102    +1     
  Branches    10951    10951           
=======================================
+ Hits        47962    47963    +1     
  Misses      14632    14632           
  Partials     2507     2507           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ksbeattie ksbeattie added the Priority:High High Priority Issue or PR label Feb 12, 2026
Copy link
Contributor

@dallan-keylogic dallan-keylogic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@blnicho
Copy link
Member Author

blnicho commented Feb 14, 2026

IDAES/examples#161 should fix the same issue in the examples repo. Once it gets merged, that should resolve the integration test failures.

@blnicho blnicho added the CI:run-integration triggers_workflow: Integration label Feb 19, 2026
@idaes-build idaes-build removed the CI:run-integration triggers_workflow: Integration label Feb 19, 2026
@blnicho blnicho merged commit 5a979c4 into IDAES:main Feb 19, 2026
95 of 138 checks passed
@blnicho blnicho deleted the remove-pkg-resources branch February 19, 2026 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority:High High Priority Issue or PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants