-
-
Notifications
You must be signed in to change notification settings - Fork 25.9k
ENH Adding variable force_alpha
to classes in naive_bayes.py
#22269
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
79 commits
Select commit
Hold shift + click to select a range
282a7dd
Adding variable alphaCorrection to classes in naive_bayes.py.
arka204 d78e17b
Splitting few lines of code.
arka204 3b79637
Merge pull request #1 from scikit-learn/master
arka204 464dc37
Merge pull request #2 from scikit-learn/master
arka204 a4429bf
Fixing problems and adding tests.
arka204 cf35eb1
Updating naive_bayes.py.
arka204 15a658f
Merge pull request #3 from scikit-learn/master
arka204 ec786b3
Merge branch 'alpha-close-or-equal-0-update' into alpha-1
arka204 4606d85
Merge pull request #5 from arka204/alpha-1
arka204 f0debb6
Merge pull request #6 from arka204/alpha-close-or-equal-0-update
arka204 dcce4a8
Checkig warnings in tests.
arka204 81d5f32
Merge branch 'alpha-close-or-equal-0-update' of https://github.com/ar…
arka204 43dfda5
Merge pull request #8 from arka204/alpha-close-or-equal-0-update
arka204 be0ebfe
Update v0.24.rst
arka204 2968400
Merge pull request #10 from scikit-learn/master
arka204 7e1b649
Merge branch 'Proposition-for-BernoulliNB-and-MultinomialNB-when-alph…
arka204 5782e6f
Merge pull request #11 from arka204/master-copy
arka204 a7337d2
Merge remote-tracking branch 'upstream/master' into Proposition-for-B…
2d16091
Fix merge
728b842
Merge remote-tracking branch 'upstream/master' into 10772-force-alpha
hongshaoyang a8209e0
Move whatsnew
hongshaoyang 44c50fd
Merge remote-tracking branch 'upstream/master' into 10772-force-alpha
hongshaoyang ded1f6e
Merge remote-tracking branch 'upstream/main' into 10772-force-alpha
hongshaoyang 2a2d8f3
Apply suggestions from code review
hongshaoyang d8f784e
Remove extra line
hongshaoyang a3897f7
Flake8
hongshaoyang 23c68dd
Apply suggestions from code review
hongshaoyang b1151d7
Merge remote-tracking branch 'upstream/main' into 10772-force-alpha
hongshaoyang 1d01c6c
Fix merge
hongshaoyang aa1d8de
use assert_warns_message
hongshaoyang 203af9e
Apply suggestions from code review
hongshaoyang cc4fda7
Merge remote-tracking branch 'upstream/main' into 10772-force-alpha
hongshaoyang 91127bc
Fix wrong variable name
hongshaoyang c4d0736
Fix test to use "with pytest.warns" instead of assert_warns_message
hongshaoyang 8964a16
Merge commit '0e7761cdc4f244adb4803f1a97f0a9fe4b365a99' into 10772-fo…
hongshaoyang e7a5f37
MAINT Adds target_version to black config (#20293)
thomasjpfan 98c0c12
Black formatting
hongshaoyang 2d9ab41
Merge remote-tracking branch 'upstream/main' into 10772-force-alpha
hongshaoyang 16af708
Apply suggestions from code review
hongshaoyang 180721f
Merge branch 'main' into 10772-force-alpha
Micky774 62d9f5b
Updated versioning and improved warning message
Micky774 9bc6f74
Updated docs to include literal value of `_ALPHA_MIN`
Micky774 0d6f2ce
Updated tests and improved `force_alpha` documentation
Micky774 28e34d9
Merge branch 'main' into 10772-force-alpha
Micky774 858c279
Merge branch 'main' into 10772-force-alpha
Micky774 39769e3
Merge branch 'main' into 10772-force-alpha
Micky774 61e5a28
Merge branch 'main' into 10772-force-alpha
Micky774 036fcac
Fixed changelog formatting
Micky774 635d403
Merge branch 'main' into 10772-force-alpha
Micky774 f2038d9
Merge branch 'main' into 10772-force-alpha
Micky774 be9fb63
Apply suggestions from code review
Micky774 5c3c7c9
Fixed tests to avoid FutureWarning
Micky774 fdef471
Merge branch '10772-force-alpha' of https://github.com/Micky774/sciki…
Micky774 87a459a
Merge branch 'main' into 10772-force-alpha
Micky774 77e0dbe
Update sklearn/tests/test_naive_bayes.py
Micky774 641036b
Merge branch '10772-force-alpha' of https://github.com/Micky774/sciki…
Micky774 0158331
Merge branch 'main' into 10772-force-alpha
Micky774 5ddd625
Updated examples with `force_alpha=True`
Micky774 699ab83
Updated tests and code for double-deprecation cycle
Micky774 91b6f3c
Merge branch 'main' into 10772-force-alpha
Micky774 41be745
Merge branch 'main' into 10772-force-alpha
Micky774 89ddbf1
Simplification of validation
Micky774 63f9fd8
Merge branch 'main' into 10772-force-alpha
Micky774 14a360f
Added private `_alpha` attribute for testing ease
Micky774 84743b6
Revert "Added private `_alpha` attribute for testing ease"
Micky774 c855059
Update sklearn/naive_bayes.py
Micky774 579f696
Merge branch 'main' into 10772-force-alpha
Micky774 782f10a
Filtered warnings in tests, fixed warning message
Micky774 30cf579
Merge branch 'main' into 10772-force-alpha
thomasjpfan 97fa75a
Merge branch 'main' into 10772-force-alpha
Micky774 f6376ff
Made quieter warning and expanded test
Micky774 764fb58
Streamlined test
Micky774 615ebdb
Updated test
Micky774 f3462d2
Apply suggestions from code review
Micky774 5828e0e
Apply suggestions from code review
Micky774 02cb3db
Merge branch 'main' into 10772-force-alpha
Micky774 15b964b
Merge branch 'main' into 10772-force-alpha
Micky774 cb2f886
Addressed review feedback
Micky774 5715086
Apply suggestions from code review
Micky774 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.