Skip to content

Power transformer - #505

Open
andrewdalpino wants to merge 5 commits into
3.1from
power-transformer
Open

andrewdalpino wants to merge 5 commits into
3.1from
power-transformer

Conversation

@andrewdalpino

Copy link
Copy Markdown
Member

Yeo-Johnnson type.

@andrewdalpino
andrewdalpino requested review from a team and a lite review from Copilot September 20, 2026 16:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

Near-zero finite values can be lost by cancellation in forward, inverse, and likelihood calculations.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 Medium severity

Open (1)
What changed in this PR

Adds a Yeo-Johnson PowerTransformer with fitting, reversal, tests, benchmarking, and documentation.

Changes:

  • Implements per-feature or fixed-lambda transformation and MLE estimation.
  • Adds unit tests and a benchmark.
  • Adds transformer documentation and navigation entries.
File Description
src/​Transformers/​PowerTransformer.php Core transformer implementation
tests/​Transformers/​PowerTransformerTest.php Functional and persistence tests
benchmarks/​Transformers/​PowerTransformerBench.php Performance benchmark
docs/​transformers/​power-transformer.md API documentation
docs/​preprocessing.md Preprocessing overview entry
mkdocs.yml Documentation navigation entry

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/Transformers/PowerTransformer.php Outdated
andrewdalpino and others added 2 commits September 20, 2026 12:42
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

This branch has not been deployed

No deployments
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