Skip to content

fix: use var_ instead of deprecated sigma_ in GaussianNB test#101

Open
bugparty wants to merge 1 commit into
ddbourgin:masterfrom
bugparty:worktree-agent-a926ff827317a137c
Open

fix: use var_ instead of deprecated sigma_ in GaussianNB test#101
bugparty wants to merge 1 commit into
ddbourgin:masterfrom
bugparty:worktree-agent-a926ff827317a137c

Conversation

@bugparty
Copy link
Copy Markdown

Summary

  • Replace all sklearn_NB.sigma_ with sklearn_NB.var_ in test_naive_bayes.py (lines 45, 51, 66), since sklearn renamed the GaussianNB attribute from sigma_ to var_
  • Fix collections.Hashable import in data_structures.py to use collections.abc.Hashable for Python 3.10+ compatibility

Test plan

  • uv run pytest numpy_ml/tests/test_naive_bayes.py --tb=short passes (1 passed)

🤖 Generated with Claude Code

sklearn renamed the GaussianNB attribute from sigma_ to var_. Also
fix collections.Hashable import for Python 3.10+ compatibility.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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