Skip to content
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

Updated naive bayers classifier #4971

Closed
wants to merge 1 commit into from

Conversation

Dynamo13
Copy link

Used pandas to import dataset sklearn for preprocessing and confusion matrix to check accuracy. The classifier is GaussianNB( lines 23-26)

Used pandas to import dataset sklearn for preprocessing  and confusion matrix to check accuracy. The classifier is GaussianNB( lines 23-26)
@vigsterkr vigsterkr closed this Mar 28, 2020
@karlnapf
Copy link
Member

It just keeps coming....

@Dynamo13
Copy link
Author

Well I need to know where I am going wrong!

@geektoni
Copy link
Contributor

@Dynamo13 the problem here is that you are not addressing any particular Shogun issue. You are just adding random code in the wrong places. If you want to work on examples, then you should have a look at our cookbook system and go from there (#3555). If you want to work on other issues, just go through our "Issue" list here on Github. Please also make sure to read our Wiki's guide about how to contribute to the project itself.

@vigsterkr
Copy link
Member

@Dynamo13 i have written you explicitly here what you are doing wrong: #4966 (comment)

this is the very last time.... the next time i'll have to report you as a spammer on github.

@vigsterkr
Copy link
Member

Well I need to know where I am going wrong!

do you really think this is the appropriate way to move forward when people are closing your PRs right away and telling you what you should consider doing? you just ignore the requests and keep on doing it...

@karlnapf
Copy link
Member

I don’t think there is any point replying to this as this is likely a group of people trying to mass spam their way into gsoc. There were reports of this at the mailing list at some point.

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.

4 participants