Skip to content

Support NaN-masked training and batch updates - #32

Open
JannisNe wants to merge 5 commits into
andremsouza:masterfrom
JannisNe:fix/masked-nan-training
Open

JannisNe wants to merge 5 commits into
andremsouza:masterfrom
JannisNe:fix/masked-nan-training

Conversation

@JannisNe

@JannisNe JannisNe commented Sep 24, 2026 •

Copy link
Copy Markdown

Summary

  • ignore missing feature values when calculating Euclidean BMUs
  • accumulate batch-training sums and counts per feature
  • preserve previous weights for features with no observations
  • add/update NaN-focused examples and regression coverage

Validation

  • poetry run pytest -q: 717 passed, 1 skipped
  • One unrelated existing failure remains in tests/test_packaging.py: pyproject.toml declares >=3.10,<4.0 while that test expects >=3.10.

Unrelated local working-tree files were not included.

Usage of AI

This PR was prepared and coded with the help of AI / Copilot.

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.

1 participant