Skip to content

Update gaussian_naive_bayes.py#7406

Merged
cclauss merged 2 commits intoTheAlgorithms:masterfrom
mvsg2:patch-1
Oct 19, 2022
Merged

Update gaussian_naive_bayes.py#7406
cclauss merged 2 commits intoTheAlgorithms:masterfrom
mvsg2:patch-1

Conversation

@mvsg2
Copy link
Copy Markdown
Contributor

@mvsg2 mvsg2 commented Oct 18, 2022

Fixes #7407

Just adding in a final metric of accuracy to declare...

Describe your change:

I have imported another module from sklearn.metrics library i.e. accuracy_score to display the final accuracy of the trained model out on the console. Also, just to make it a little propped up, I also imported the time library and stopped the code-execution at two points in the middle (this won't affect the final accuracy of the model in any way).

  • Add an algorithm?
  • Fix a bug or typo in an existing algorithm?
  • Documentation change?

Checklist:

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized.
  • I know that pull requests will not be merged if they fail the automated tests.
  • This PR only changes one algorithm file. To ease review, please open separate PRs for separate algorithms.
  • All new Python files are placed inside an existing directory.
  • All filenames are in all lowercase characters with no spaces or dashes.
  • All functions and variable names follow Python naming conventions.
  • All function parameters and return values are annotated with Python type hints.
  • All functions have doctests that pass the automated testing.
  • All new algorithms have a URL in its comments that points to Wikipedia or other similar explanation.
  • If this pull request resolves one or more open issues then the commit message contains Fixes: #{$ISSUE_NO}.

mvsg2 and others added 2 commits October 18, 2022 22:44
Just adding in a final metric of accuracy to declare...
@ZeroDayOwl ZeroDayOwl mentioned this pull request Oct 19, 2022
14 tasks
# Gaussian Naive Bayes
nb_model = GaussianNB()
nb_model.fit(x_train, y_train)
time.sleep(2.9)
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.

Why are we waiting for 3 seconds?!? Comments needed to be added to explain such things.

@cclauss cclauss added hacktoberfest hacktoberfest-accepted Accepted to be counted towards Hacktoberfest labels Oct 19, 2022
@cclauss cclauss merged commit 80ff25e into TheAlgorithms:master Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest hacktoberfest-accepted Accepted to be counted towards Hacktoberfest

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants