Skip to content

[ENH] allow returning empty variable lists in funcs#894

Merged
solegalli merged 30 commits intomainfrom
allow_empty_var_lists_in_funcs
Feb 11, 2026
Merged

[ENH] allow returning empty variable lists in funcs#894
solegalli merged 30 commits intomainfrom
allow_empty_var_lists_in_funcs

Conversation

@solegalli
Copy link
Collaborator

@solegalli solegalli commented Feb 8, 2026

I am changing the functionality of the variable that finds numerical and categorical variables when the user enters a list. This might have knock-on effects downstream on other transformers, particularly on the selectors that use the prediction module. Need to double check and update the documentation when suitable.

@solegalli solegalli force-pushed the allow_empty_var_lists_in_funcs branch from 6dabd6b to b07b073 Compare February 8, 2026 15:37
@codecov
Copy link

codecov bot commented Feb 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.23%. Comparing base (ec04aa4) to head (dde61c3).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #894   +/-   ##
=======================================
  Coverage   98.22%   98.23%           
=======================================
  Files         113      113           
  Lines        4846     4867   +21     
  Branches      770      777    +7     
=======================================
+ Hits         4760     4781   +21     
  Misses         55       55           
  Partials       31       31           

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

@solegalli solegalli merged commit 48b3938 into main Feb 11, 2026
14 checks passed
@solegalli solegalli deleted the allow_empty_var_lists_in_funcs branch February 11, 2026 15:04
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

Comments