-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
Phase-1 Contributions for Analyzer #12
base: develop
Are you sure you want to change the base?
Conversation
ashish493
commented
Jun 12, 2020
- Added dataset
- Added models for predictions
- Added Notebooks for analysis
created analyzer app
@hardlyhuman Can u review the changes? |
Why do we have jupyter notebook files here? Also make necessary changes views.py of analyzer app |
Sure, I will make the changes for the analyzer app. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hardlyhuman Can u review these few changes?
try: | ||
input_data = self.preprocessing(input_data) | ||
prediction = self.predict(input_data) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And here, we will make changes once we decide a prpoer format of input data
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what's the point of having 'self.predict(input_data)'?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, it doesn't have any significance at the moment as we haven't created the input serializers. So, I will remove and then create a separate app for prediction.
from rest_framework.renderers import JSONRenderer | ||
|
||
class EndpointSerializer(serializers.ModelSerializer): | ||
read_only_fields = ("model_name", "param1", "param2", "param3") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will modify it as soon as the necessary changes are made