-
-
Notifications
You must be signed in to change notification settings - Fork 216
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
Vulnerable Cancer Patients #536
Conversation
Our team will soon review your PR. Thanks @CoderOMaster :) |
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.
Not every issue will mark as Hard, if it goes like this every issue should be of Hard category then.
- Project folder name should be same as the issue name with no underscore.
- Put some EDA before going in to the models. Such as Predicted datapoints plot, intensity distribution plot etc.
- Make the README a bit attractive.
I understand sir. |
@abhisheks008 |
| Neural Network | 0.67 | | ||
| Logistic Regression| 0.71 | | ||
|
||
**NOTE: No graphs were plotted** |
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.
You have plotted graphs. Remove this and add Visualization heading under that put the graph of intensity distribution. After that Models Applied will come.
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 exactly am I supposed to do? I don't get it please
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.
You have written no graphs were plotted that's why I told you to remove that line and make the changes accordingly.
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 have done this too..Please merge it
|
||
**ROBERTA MODEL IN TENSORFLOW AND PYTORCH WERE PARTIALLY TRAINED DUE TO RESOURCE LIMITATIONS; RESULTS FROM 3 EPOCHS WILL BE PROVIDED** | ||
|
||
**IMAGE** |
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.
This is creating some styling problem, put <br>
over here.
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 removed that paragraph since it was obvious in the code that I took 3 epochs
![Alt Text](./Images/confusion_matrix.png) | ||
|
||
![Alt Text](./Images/intensity.png) | ||
|
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.
Missing Conclusion.
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.
Conclusion as in?
I have the result plotted
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.
After finishing a project don't you do some conclusion, this this things we can conclude or just left the project without putting any conclusion?
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.
Done ✅
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.
Approved.
@CoderOMaster
Thank you |
Pull Request for ML-Crate 💡
Issue Title: Issue No 498
Please Mark this as hard:)