Skip to content

Conversation

@t0mdavid-m
Copy link
Member

@t0mdavid-m t0mdavid-m commented Jan 23, 2026

Summary by CodeRabbit

  • New Features

    • Enhanced parameter filtering with improved advanced settings toggle for better control over which parameters are displayed based on inclusion and exclusion rules.
  • Refactor

    • Optimized parameter visibility handling and display logic for improved performance.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 23, 2026

📝 Walkthrough

Walkthrough

This change enhances parameter filtering logic in StreamlitUI.py by introducing suffix-based pattern matching for parameter keys, centralizing exclusion rules, and implementing a two-phase filtering approach. Parameters now carry a "non_included" visibility flag, enabling conditional display based on advanced mode and inclusion patterns.

Changes

Cohort / File(s) Summary
Parameter Filtering Enhancement
src/workflow/StreamlitUI.py
Added _matches_parameter() helper for suffix-based key matching; centralized excluded_keys list (log, debug, threads, no_progress, force, version, test); implemented two-phase filtering: valid_keys computation and included_keys subset derivation; augmented parameters with "non_included" flag for conditional visibility; updated rendering logic in subsection and non-subsection paths to respect visibility flag based on advanced toggle (+55/-26, High review effort)

Poem

🐰 With patterns and filters, we hop through the keys,
Suffix-matching magic puts parameters at ease.
Two phases of wisdom, inclusion precise,
Non-included flags sparkle—a visibility device!
Streamlit UI blooms with advanced-mode grace.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'extend advanced section, fix matching' directly reflects the main changes: extending advanced parameter handling and improving parameter matching logic with the new _matches_parameter function.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

2 participants