-
Notifications
You must be signed in to change notification settings - Fork 1
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
Submission: 6: New Taipei City Real Estate Value Prediction #6
Comments
Data analysis review checklistReviewer: mahdiheydarConflict of interest
Code of Conduct
General checks
Documentation
Code quality
Reproducibility
Analysis report
Estimated hours spent reviewing: 1 hourReview Comments:Really great job! your report was very easy to follow and concise. There are only a couple minor pointers that I can offer:
Again, great job, I wish you guys all the best on your final project and exams! AttributionThis was derived from the JOSE review checklist and the ROpenSci review checklist. |
Data analysis review checklistReviewer:Conflict of interest
Code of Conduct
General checks
Documentation
Code quality
Reproducibility
Analysis report
Estimated hours spent reviewing: 1 hourReview Comments:Interesting project! Here are some things you guys did well or could improve on:
AttributionThis was derived from the JOSE review checklist and the ROpenSci review checklist. |
Data analysis review checklistReviewer:Conflict of interest
Code of Conduct
General checks
Documentation
Code quality
Reproducibility
Analysis report
Estimated hours spent reviewing: 1.5 HourReview Comments:
AttributionThis was derived from the JOSE review checklist and the ROpenSci review checklist. |
Data analysis review checklistReviewer:Conflict of interest
Code of Conduct
General checks
Documentation
Code quality
Reproducibility
Analysis report
Estimated hours spent reviewing: 2hrsReview Comments:Please provide more detailed feedback here on what was done particularly well, and what could be improved. It is especially important to elaborate on items that you were not able to check off in the list above. 1. Overall the project is great and easy for reader to know the analyzing processes. The docker instruction is clear and the docker makefile works well. Also the docker image can be pulled successfully. Files are mostly well-organized.2. Makefile issuesThe makefile does not work properly. Data folder does not exist and the data cannot be generated accordingly. 3. Path nameThe makefile does not work well so I can only see your report from "Prediction_of_Real_Estate_Value.ipynb" file. For loading the data, it would be better to use relative path instead of the absolute path since the current path does not work on any other machines except the project author's . 4. Add more explanations to methodsThe method of selecting predictors using forward selection is great and robust. Also the visualization can let people easily know the result. Since I have statistics background, it is ok for me to understand those methods such as "forward selection". It would be even better to add some text explanations about this "forward selection" technique as well as the metric - "Mallows' CP" so that people without any relevant knowledge can more easily understand the whole process. 5. Function input clarificationIn split data function documentation, it looks that the input can only be a dataframe not a dataset (i.e. csv or excel files), so the description for input "dataset" should be fixed accordingly instead of allowing it to be either a dataset or a dataframe. 6. Function documentations incomplete (very tiny thing to suggest)Function documentations are not well formatted. Some functions are using "#" while others using "". Also some function documentations are missing parts. But these does not harm a lot. 7. Function name convention (very tiny thing to suggest)It looks all functions are following snake case naming convention, so all functions are expected to have lower case letters. AttributionThis was derived from the JOSE review checklist and the ROpenSci review checklist. |
Hi, did you download the docker image? Me and three other group members can run the makefile without error on the docker container |
Submitting authors: @asmdrk @AaronMKk @ZiyueChloeZhang @mcloses
Repository: https://github.com/DSCI-310/DSCI-310-Group-6
Abstract/executive summary:
In this project, we build a regression model that estimates the price per unit of area of houses given the transaction date, the age of the house, the distance to the nearest MRT station, the number of convenience stores, the latitude and the longitude in the Sindian District of New Taipei City. Predictors are chosen through forward selection. Out of the two models we build, we use ANOVA test to pick the model with interaction as the final model. RMSE is used as the evaluation metric for this model.
Editor: @ttimbers
Reviewer: @eahn01 @YuYT98 @luckyberen @mahdiheydar
The text was updated successfully, but these errors were encountered: