Skip to content
Permalink

Comparing changes

Choose two branches to see what鈥檚 changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: kevin0315/openml-python
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main
Choose a base ref
...
head repository: openml/openml-python
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: main
Choose a head ref
Checking mergeability… Don鈥檛 worry, you can still create the pull request.
  • 3 commits
  • 15 files changed
  • 9 contributors

Commits on Jun 3, 2026

  1. [ENH] Reduce complexity of run_flow_on_task func (openml#1596)

    ## Summary
    
    This PR refactors `run_flow_on_task`, which had grown to ~160 lines with
    high cyclomatic complexity, by extracting small helper functions with
    clear, single responsibilities. The main function is now a readable
    orchestrator with clearly defined steps.
    
    ---
    
    ## Changes
    
    ### Extracted helper functions
    
    * **`_validate_flow_and_task_inputs`**
      Handles input validation and backward-compatible argument handling
    
    * **`_sync_flow_with_server`**
      Synchronizes the flow with the server and checks for duplicate runs
    
    * **`_prepare_run_environment`**
      Prepares environment information and run tags
    
    * **`_create_run_from_results`**
      Builds the `OpenMLRun` object from execution results
    
    ### Internal structure improvements
    
    * Introduced a `_RunResults` `NamedTuple` to bundle execution outputs
    (`data_content`, `trace`, `evaluations`) and reduce long parameter lists
    
    ---
    
    ## Type Safety Improvements
    
    * Replaced `assert` statements with explicit `ValueError` / `TypeError`
    exceptions
    * Added type guards before accessing attributes that may be `None`
    * Made boolean parameters keyword-only where appropriate
    
    ---
    
    Fixes openml#1580
    
    ---------
    
    Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>
    Co-authored-by: Franz Kir谩ly <fkiraly@gcos.ai>
    Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
    Co-authored-by: Armaghan Shakir <raoarmaghanshakir040@gmail.com>
    4 people authored Jun 3, 2026
    Configuration menu
    Copy the full SHA
    3486b2a View commit details
    Browse the repository at this point in the history
  2. [ENH] V1 -> V2 Migration : Runs (openml#1616)

    fixes openml#1624
    
    ---------
    
    Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>
    Co-authored-by: Simon Blanke <simon.blanke@yahoo.com>
    Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
    Co-authored-by: geetu040 <raoarmaghanshakir040@gmail.com>
    Co-authored-by: Satvik Mishra <112589278+satvshr@users.noreply.github.com>
    Co-authored-by: Matthias Feurer <lists@matthiasfeurer.de>
    Co-authored-by: Pieter Gijsbers <p.gijsbers@tue.nl>
    Co-authored-by: Franz Kir谩ly <fkiraly@gcos.ai>
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    9 people authored Jun 3, 2026
    Configuration menu
    Copy the full SHA
    c631b28 View commit details
    Browse the repository at this point in the history

Commits on Jun 4, 2026

  1. [ENH] V1 -> V2 Migration - Flows (module) (openml#1609)

    Fixes openml#1601 
    
    ---------
    
    Signed-off-by: Omswastik-11 <omswastikpanda11@gmail.com>
    Co-authored-by: geetu040 <raoarmaghanshakir040@gmail.com>
    Co-authored-by: Simon Blanke <simon.blanke@yahoo.com>
    Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
    Co-authored-by: Satvik Mishra <112589278+satvshr@users.noreply.github.com>
    Co-authored-by: Matthias Feurer <lists@matthiasfeurer.de>
    Co-authored-by: Pieter Gijsbers <p.gijsbers@tue.nl>
    Co-authored-by: Franz Kir谩ly <fkiraly@gcos.ai>
    Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
    9 people authored Jun 4, 2026
    Configuration menu
    Copy the full SHA
    fabbc28 View commit details
    Browse the repository at this point in the history
Loading