Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: implemented imputation for missing values #140

Merged
merged 2 commits into from
Jun 2, 2022
Merged

Conversation

Sparkier
Copy link
Owner

Basic implementation of nearest neighbor imputation for missing values.

@Sparkier Sparkier requested a review from onc April 15, 2022 09:43
@Sparkier Sparkier self-assigned this Apr 15, 2022
Comment on lines 12 to 28
const dataFinal = data.axes.find((a) => a.name === axis.name);
if (dataFinal === undefined) {
return;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do you mean dataForAxis? dataFinal sounds weird.

axes: AxisDescriptor[],
includeCategorical = true
): number | string | undefined {
const distances: Record<number, number> = {};
Copy link
Collaborator

Choose a reason for hiding this comment

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

I dont really understand what the distances are for tbh.

Alex added 2 commits June 1, 2022 18:52
Basic implementation of nearest neighbor imputation for missing values.
@Sparkier Sparkier merged commit 6bfa62d into main Jun 2, 2022
@Sparkier Sparkier deleted the imputation branch June 2, 2022 07:07
@github-actions
Copy link

github-actions bot commented Jun 8, 2022

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants