Skip to content

Added project with various techniques applied to Kaggle graduate admissions data#115

Closed
vishxm wants to merge 1 commit intogimseng:masterfrom
vishxm:vishxm-do
Closed

Added project with various techniques applied to Kaggle graduate admissions data#115
vishxm wants to merge 1 commit intogimseng:masterfrom
vishxm:vishxm-do

Conversation

@vishxm
Copy link
Copy Markdown

@vishxm vishxm commented Oct 4, 2020

Reference Issues/PRs

What does this implement/fix? Explain your changes.

I fixed and made the necessary changes which make this PR in accordance with the other projects.

Any other comments?

After this PR is merged, I need to update the README.md to change the path of the hyperlink in the Solution heading.

@gimseng
Copy link
Copy Markdown
Owner

gimseng commented Oct 5, 2020

@vishxm Could you link or refer to what other projects you have in mind?
If this is on a new project, please fill in the details of the PR template. Thanks !

@vishxm
Copy link
Copy Markdown
Author

vishxm commented Oct 5, 2020

The changes were made on an earlier PR. Mr Khalsa suggested some changes to be made in project description. I made the necessary ones.

@gimseng
Copy link
Copy Markdown
Owner

gimseng commented Oct 5, 2020

@vishxm yes, but do link to that particular PR, even if its closed. And have a brief summary/description of what the other PR is. This helps to keep a clean history and a readable thread.

@vishxm
Copy link
Copy Markdown
Author

vishxm commented Oct 5, 2020

My bad! Here is the link to the previous PR #110. The description can be found in the README of the newly added directory. Link to that is https://github.com/vishxm/99-ML-Learning-Projects/blob/vishxm-do/008/README.md

@gimseng gimseng changed the title readme added Added project with regression (follow up to #110) Oct 5, 2020
@gimseng
Copy link
Copy Markdown
Owner

gimseng commented Oct 5, 2020

@vishxm Thanks for the reply. A few questions/suggestions/comments:

  1. Does this PR (and PR Added project with regression #110) resolve anything in the issue ticket? We prefer that you create your new project ideas based on an existing issue ticket. This allows discussions to happen, and we could provide suggestions and comments on the idea/project scope, especially on how it fits into the general goals of the repo.

  2. I think your data and solution folders are flipped.

  3. Please provide link to your kaggle data source in the readme.md in either (or both) exercise or data. Please do not have a readme.md' in your project root folder, perhaps delegate the information to either data/exercise/solutions`.

  4. Please rename your project folder name to other placeholder name, to avoid conflicts.

  5. Please describe/summarize the data either in the readme.md in either (or both) exercise or data.

  6. I am a little confused about this project/exercise. Is it a regression problem or a classification problem? Moreover, is it a supervised or unsupervised problem (e.g. clustering?)? Related to 1 above, this is another reason to discuss this in an issue ticket, where many people can contribute ideas/discussions. I think its ok to have an interesting dataset and perform a few types of ML techniques. But I'd like a more detailed descriptions of why different types of techniques are required.

@gimseng gimseng changed the title Added project with regression (follow up to #110) Added project with various techniques applied to Kaggle graduate admissions data Oct 5, 2020
@vishxm vishxm closed this Jul 14, 2024
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.

2 participants