-
Notifications
You must be signed in to change notification settings - Fork 0
Comparing changes
Open a pull request
base repository: rupeshbharambe24/dissectML
base: v0.1.2
head repository: rupeshbharambe24/dissectML
compare: v0.1.3
- 13 commits
- 24 files changed
- 1 contributor
Commits on Apr 7, 2026
-
Configuration menu - View commit details
-
Copy full SHA for 683ebae - Browse repository at this point
Copy the full SHA 683ebaeView commit details -
Configuration menu - View commit details
-
Copy full SHA for 3767291 - Browse repository at this point
Copy the full SHA 3767291View commit details
Commits on Apr 26, 2026
-
Configuration menu - View commit details
-
Copy full SHA for 40046ca - Browse repository at this point
Copy the full SHA 40046caView commit details -
Configuration menu - View commit details
-
Copy full SHA for b3291d5 - Browse repository at this point
Copy the full SHA b3291d5View commit details
Commits on Jun 21, 2026
-
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.
Configuration menu - View commit details
-
Copy full SHA for ce5f94f - Browse repository at this point
Copy the full SHA ce5f94fView commit details -
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.
Configuration menu - View commit details
-
Copy full SHA for 1375ea1 - Browse repository at this point
Copy the full SHA 1375ea1View commit details -
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'.
Configuration menu - View commit details
-
Copy full SHA for 32f539e - Browse repository at this point
Copy the full SHA 32f539eView commit details -
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.
Configuration menu - View commit details
-
Copy full SHA for d86656c - Browse repository at this point
Copy the full SHA d86656cView commit details -
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.
Configuration menu - View commit details
-
Copy full SHA for 60586b7 - Browse repository at this point
Copy the full SHA 60586b7View commit details -
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.
Configuration menu - View commit details
-
Copy full SHA for 705c662 - Browse repository at this point
Copy the full SHA 705c662View commit details -
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.)
Configuration menu - View commit details
-
Copy full SHA for 66e5470 - Browse repository at this point
Copy the full SHA 66e5470View commit details -
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.
Configuration menu - View commit details
-
Copy full SHA for 0d680ee - Browse repository at this point
Copy the full SHA 0d680eeView commit details -
Configuration menu - View commit details
-
Copy full SHA for 6ed5d0c - Browse repository at this point
Copy the full SHA 6ed5d0cView commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine: git diff v0.1.2...v0.1.3