Skip to content

Add PedAnovaImportanceEvaluator - #57

Merged
y0z merged 37 commits into
mainfrom
kAIto47802/add-ped-anova
Feb 13, 2026
Merged

y0z merged 37 commits into
mainfrom
kAIto47802/add-ped-anova

Conversation

@kAIto47802

@kAIto47802 kAIto47802 commented Feb 10, 2026 •

Copy link
Copy Markdown
Collaborator

Motivation & Description of the changes

  • Add PedAnovaImportanceEvaluator

Note

Currently, the params argument is not supported. We plan to add support it when we implement condPED-ANOVA:

Note

To keep this PR minimal, I did not modify fanova.rs in this PR. We can address it in a follow-up PR.

@kAIto47802
kAIto47802 marked this pull request as draft February 10, 2026 08:10
@kAIto47802
kAIto47802 marked this pull request as ready for review February 13, 2026 08:15

@y0z y0z left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@y0z
y0z merged commit 283a9ae into main Feb 13, 2026
6 checks passed
@y0z
y0z deleted the kAIto47802/add-ped-anova branch February 13, 2026 09:29
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