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.

Sign up for GitHub

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

Jump to bottom

FMLearn implementation using K Nearest Neighbor #16

Merged
merged 15 commits into from
Jul 20, 2020
Merged

FMLearn implementation using K Nearest Neighbor #16

merged 15 commits into from
Jul 20, 2020

Conversation

mukeshmk
Copy link
Owner

FMLearn implementation using the K Nearest Neighbor algorithm, specifically sci-kit learn's KNeighborsClassifier.

This PR also introduces additional features like:
#12 force new model to be trained once data has been reloaded.
#13 throw error if the data is not loaded before training.
#14 check the shape of the input data frame.
#15 force retrain of the model if the model is older than a set time frame.

@mukeshmk
modified fmlearn to predeict the dataset isntead of algo
1d73799
@mukeshmk
WIP: added base class for FMLearn algorithm
6239317
@mukeshmk
added TODOs and test function to fmlearn class
446d426
@mukeshmk
WIP: updated the app.py with fmlearn class test call
fe73f9d
@mukeshmk
added JSON constants to the constants.py file changed code
f575fe4
@mukeshmk
fixed issue with fmlearn datatime.now()
353cd48
@mukeshmk
added encoders to the class for later use while encoding inputs for p…
3efdfbf 
…redict()
@mukeshmk
WIP: added test api for fmlearn predict()
fe1d8b4
@mukeshmk
moved around API's as per there usage
18ac17a
@mukeshmk
fixes #14: check the shape of the input dataframe for predict
10881bf
@mukeshmk
fixes #13: throw error if the data is not loaded before training
6e2db30
@mukeshmk
WIP: upated return value of predict_fmlearn() with predicted df
b683053
@mukeshmk
fixed issue where proper resposne not being sent to sklearn
9ecec12
@mukeshmk
fixes #12: force new model to be trained once data has been reloaded
5165f6b
@mukeshmk
fixes #15: force reload and retrain if the set no of new records have…
076e5c1 
… been added
@mukeshmk mukeshmk added the enhancement New feature or request label Jul 20, 2020
@mukeshmk mukeshmk self-assigned this Jul 20, 2020
@mukeshmk mukeshmk merged commit 09060dc into master Jul 20, 2020
@mukeshmk mukeshmk deleted the wip-fmlearn branch July 20, 2020 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mukeshmk mukeshmk

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mukeshmk