Skip to content

Add a skrub config manager - #1427

Merged
Vincent-Maladiere merged 18 commits into
skrub-data:mainfrom
Vincent-Maladiere:add_skrub_options
Jun 23, 2025
Merged

Vincent-Maladiere merged 18 commits into
skrub-data:mainfrom
Vincent-Maladiere:add_skrub_options

Conversation

@Vincent-Maladiere

Copy link
Copy Markdown
Member

Fixes #1377

@Vincent-Maladiere
Vincent-Maladiere marked this pull request as draft June 3, 2025 14:24
@Vincent-Maladiere
Vincent-Maladiere marked this pull request as ready for review June 3, 2025 16:12
Comment thread skrub/_config.py Outdated
Comment thread skrub/_config.py
Comment thread skrub/_config.py
Comment thread skrub/_config.py Outdated
Comment thread skrub/_config.py
Comment thread skrub/_reporting/_table_report.py Outdated
Comment thread skrub/_reporting/_table_report.py Outdated
Comment thread skrub/_reporting/_patching.py
Comment thread skrub/_reporting/tests/test_patch_display.py Outdated
Comment thread skrub/tests/test_config.py Outdated
Comment thread skrub/_config.py Outdated

@rcap107 rcap107 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

looks good to me! thanks a lot @Vincent-Maladiere

@Vincent-Maladiere
Vincent-Maladiere merged commit f1050f6 into skrub-data:main Jun 23, 2025
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.

Adding a skrub global config

3 participants