Skip to content
Permalink

Comparing changes

Choose two branches to see what’s 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: rupeshbharambe24/dissectML
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v0.1.2
Choose a base ref
...
head repository: rupeshbharambe24/dissectML
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v0.1.3
Choose a head ref
  • 13 commits
  • 24 files changed
  • 1 contributor

Commits on Apr 7, 2026

  1. Configuration menu
    Copy the full SHA
    683ebae View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    3767291 View commit details
    Browse the repository at this point in the history

Commits on Apr 26, 2026

  1. Configuration menu
    Copy the full SHA
    40046ca View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    b3291d5 View commit details
    Browse the repository at this point in the history

Commits on Jun 21, 2026

  1. fix(battle): remove silent n_iter=5 cap in hyperparameter tuner

    RandomizedSearchCV was hard-capped to min(n_iter, 5), so 'tuned' mode
    ignored the user's n_iter (default 20) and the documented n_iter=30
    example. RandomizedSearchCV already self-limits when the parameter grid
    has fewer combinations than n_iter, so the cap was unnecessary and only
    reduced tuning quality.
    rupeshbharambe24 committed Jun 21, 2026
    Configuration menu
    Copy the full SHA
    ce5f94f View commit details
    Browse the repository at this point in the history
  2. docs: correct model catalog counts to 19 classifiers + 19 regressors

    The catalog defines 38 models (19 classifiers + 19 regressors), not the
    36 (19 + 17) stated in the docs. The SVC/SVR entries are each split into
    linear + rbf variants and HuberRegressor was added; the planned
    HistGradientBoosting regressor was never included. Updated README, PLAN,
    CHANGELOG, and the docs site to match the implementation.
    rupeshbharambe24 committed Jun 21, 2026
    Configuration menu
    Copy the full SHA
    1375ea1 View commit details
    Browse the repository at this point in the history
  3. refactor(viz): rename INSIGHTML_TEMPLATE to DISSECTML_TEMPLATE

    Last leftover of the old InsightML name in shipped code. The public
    symbol is renamed to DISSECTML_TEMPLATE; INSIGHTML_TEMPLATE is kept as a
    deprecated alias (same object) and remains exported so existing imports
    keep working. The Plotly template registration key was already
    'dissectml'.
    rupeshbharambe24 committed Jun 21, 2026
    Configuration menu
    Copy the full SHA
    32f539e View commit details
    Browse the repository at this point in the history
  4. refactor(report): remove dead report/sections module

    The build_*_section functions in report/sections/ were never imported by
    the live renderer (html_renderer.render_html_report builds its own
    _section_* helpers); only tests/report/test_sections.py referenced them.
    This dead duplicate was also the sole source of two reported
    inconsistencies: a VIF>5 threshold (live code uses >=10 everywhere) and
    attribute-style leakage access (live code correctly uses dict access on
    the LeakageWarning TypedDict). Removed the module and its test.
    rupeshbharambe24 committed Jun 21, 2026
    Configuration menu
    Copy the full SHA
    d86656c View commit details
    Browse the repository at this point in the history
  5. cleanup: drop dead _MNAR_IMPUTER, fix misleading InsightPipeline docs

    - Remove the unused _MNAR_IMPUTER constant in preprocessing.py (imputer
      selection is decided by separate logic; the constant was never read).
    - Correct InsightPipeline docstrings/error: the class docstring claimed it
      is 'used internally by dml.analyze()', but analyze() calls the stages
      directly and never touches this class. run() only wires Stage 1 (EDA)
      and is a documented stub; the error now points users to dml.analyze()
      for the full pipeline instead of implying it is unavailable.
    rupeshbharambe24 committed Jun 21, 2026
    Configuration menu
    Copy the full SHA
    60586b7 View commit details
    Browse the repository at this point in the history
  6. fix(battle): make tuned model scores fully consistent

    Previously _tune_one overwrote only the primary metric (accuracy/r2) with
    the RandomizedSearchCV score and kept the rest of the metrics, the std
    values, and the OOF predictions from the untuned model. The leaderboard
    then mixed metrics from two different CV schemes, and Stage-4 comparisons
    (error analysis, ROC, significance) ran on stale, un-tuned OOF predictions.
    
    Extract the runner's CV evaluation into a shared _evaluate_pipeline() and
    re-score the tuned estimator through it on the battle's CV folds, so every
    metric, std, and the OOF predictions come from one consistent scheme and
    stay comparable with the untuned models in the leaderboard.
    rupeshbharambe24 committed Jun 21, 2026
    Configuration menu
    Copy the full SHA
    705c662 View commit details
    Browse the repository at this point in the history
  7. docs: update iml alias to dml in DissectML.html code snippets

    The market-research page still used the pre-rename 'import dissectml as
    iml' alias in its API examples. The alias name is arbitrary so the
    snippets ran fine, but they contradicted the documented 'dml' convention.
    Updated all four occurrences for consistency. (File is excluded from the
    package build.)
    rupeshbharambe24 committed Jun 21, 2026
    Configuration menu
    Copy the full SHA
    66e5470 View commit details
    Browse the repository at this point in the history
  8. fix(viz): prevent clipped axis labels on horizontal bar charts

    Charts inherited the theme's fixed left margin (l=60), so long y-axis
    tick labels — model names on the metric-comparison bar chart, and feature/
    category names elsewhere — were clipped at the left edge. Enable Plotly
    automargin on both axes in the theme template so the margin auto-expands
    to fit labels, and set it explicitly on the metric_bar_chart y-axis.
    rupeshbharambe24 committed Jun 21, 2026
    Configuration menu
    Copy the full SHA
    0d680ee View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    6ed5d0c View commit details
    Browse the repository at this point in the history
Loading