Skip to content

Fix suggest_transform docstring to match POWER fallback logic #90

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
kgdunn wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Fix suggest_transform docstring to match POWER fallback logic #90

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
3cbb0e4
plan: fix suggest_transform docstring to match POWER fallback logic
claude Jun 12, 2026
c70abbb
Fix suggest_transform docstring: POWER fallback, not milder skew
claude Jun 12, 2026
1cf2216
Bump version to 0.22.1
claude Jun 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ build-backend = "hatchling.build"

[project]
name = "scorepilot"
version = "0.22.0"
version = "0.22.1"
description = "Web-based tool for PCA/PLS model analysis in chemometrics."
readme = "README.md"
requires-python = ">=3.12"
Expand Down
8 changes: 5 additions & 3 deletions src/scorepilot/core/profiling.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -142,9 +142,11 @@ def sequence(series: pd.Series) -> list[float | None]:
def suggest_transform(summary: VariableSummary) -> TransformKind:
"""Suggest a transform from a variable's shape.

A strongly right-skewed, strictly positive variable with a wide dynamic range
is a candidate for a log transform; a milder skew suggests a signed power
(root). Otherwise no transform is suggested.
A strongly right-skewed (skewness > 1.0), strictly positive variable with a
wide dynamic range (min/max ratio > 20) is a candidate for a log transform.
A strongly right-skewed variable that is not strictly positive or lacks a
wide dynamic range falls back to a signed power (root). Otherwise no
transform is suggested.
"""
if summary.column_type is not ColumnType.QUANTITATIVE or summary.skewness is None:
return TransformKind.NONE
Expand Down
Loading