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

Desu Venkata Manikanta(203174002)- All files added #19

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Desu Venkata Manikanta(203174002)- All files added #19

wants to merge 10 commits into from

Conversation

Mani9550
Copy link

No description provided.

@Mani9550 Mani9550 changed the title Added 4 tasks and let me know whether any more changes required Desu Venkata Manikanta(203174002)- All files added Oct 27, 2021
@Karrthik-Arya
Copy link
Contributor

All answers for task-1 are correct. For task-2 the graphs and plots have been correctly made but the model and algorithm part is incomplete. For task-3 some graphs have been wrongly implemented. Underscores are missing from some column names. Also, the part for making separate models for sepal and petal data is incomplete. For task-4 distance has been correctly calculated but the rest is incomplete. One more thing you can do is make separate PRs for each of the tasks by making branches in the repo. This way you will be able to get different PRs for different tasks for hacktoberfest and it will also be much more organized.

@Mani9550 Mani9550 closed this Oct 28, 2021
@Mani9550 Mani9550 reopened this Oct 28, 2021
@Mani9550
Copy link
Author

Closing this pull request and it has only MCQ task updated file

@Mani9550 Mani9550 closed this Oct 28, 2021
@Mani9550
Updated Task-4 code as per your comments
b24c886 
Added One pull request for each task. Please merge the PR
@Mani9550
Updated the code as per your comments
80d83a1 
Added one pull request for one task. please merge the PR
@Mani9550
Rename Classification_Task3_203174002 (1).ipynb to Classification_Tas…
458d386 
…k3_203174002.ipynb
@Mani9550
Updated the code as per your comments
ac77d68
@Mani9550
Rename Linear_Regression_Task2_203174002 (1).ipynb to Linear_Regressi…
fc8aa85 
…on_Task2_203174002.ipynb

Created one pull request for each question. Please merge the PR
@Mani9550 Mani9550 reopened this Oct 28, 2021
@Mani9550
Copy link
Author

Reopend and This PR has only MCQ file. Please merge PR

@Karrthik-Arya
Copy link
Contributor

This still contains files for all the 3 tasks.

@Mani9550
Copy link
Author

Mani9550 commented Nov 1, 2021

Hi sir,
I tried multiple times sending the separate PR for each task. But, I don't know where I am missing. Please help me sir

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
hacktoberfest-accepted
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Mani9550 @Karrthik-Arya