Skip to content

Fix missing configs in pip install and unregistered OmegaConf resolvers in tests#341

Merged
gbg141 merged 2 commits into
mainfrom
guille/configs_and_tests
Jun 2, 2026
Merged

Fix missing configs in pip install and unregistered OmegaConf resolvers in tests#341
gbg141 merged 2 commits into
mainfrom
guille/configs_and_tests

Conversation

@gbg141

@gbg141 gbg141 commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

Changes

  • pyproject.toml: Added "configs" to [tool.setuptools.packages.find] and a [tool.setuptools.package-data] entry with configs = ["**/*.yaml"] so the Hydra config files are bundled when installing via pip install git+...
  • test/_utils/simplified_pipeline.py: Replaced a stale, partial set of manual OmegaConf.register_new_resolver(...) calls with a single register_all_resolvers() call
  • test/utils/test_config_resolvers.py: Added register_all_resolvers() to setup_method before hydra.initialize() so all custom resolvers are available when Hydra composes configs

Fixes

  • configs/ directory missing after pip install git+...
  • omegaconf.errors.UnsupportedInterpolationType: Unsupported interpolation type define_task_level in test/pipeline/test_pipeline.py
  • omegaconf.errors.UnsupportedInterpolationType: Unsupported interpolation type set_preserve_edge_attr in test/utils/test_config_resolvers.py

@codecov

codecov Bot commented Jun 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.06%. Comparing base (76c82e2) to head (c316049).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #341      +/-   ##
==========================================
- Coverage   94.06%   94.06%   -0.01%     
==========================================
  Files         207      206       -1     
  Lines        8730     8726       -4     
==========================================
- Hits         8212     8208       -4     
  Misses        518      518              

☔ 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.

@gbg141 gbg141 merged commit b9bbdb3 into main Jun 2, 2026
6 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.

1 participant