salva-u
commented
11 months ago - Mention we do not need license for Open Data/Public Dataset " @atabak-alishiri @farrandi For the license part, I did a little bit research, your database and also article that use this database is funded by some organization and put it as the free PMC (PubMedCentral) article, which freely available for public, and I found the disclaimer on this "https://pubmed.ncbi.nlm.nih.gov/disclaimer/". I don't know how to correctly provided it in license.md, may be you create a section for the data you used and specify the disclaimer of free PMC over there (or better ask tiff on how exactly you should do this)." -put your names as authors under your project name in the report. @rbouwer
- hide the imports @rbouwer
- add some EDA plots to the report to show distribution by target @rbouwer
- make the plots look better and add some writing to go around it @salva-u
- Separate Limitations to a new header @rbouwer
- Address this "In README.md about the paragraph is concatenate in github, you should split it.(I understand that it will cause from error on markdown, may be you should give it two extra white line to separate it)" @farrandi
- Add makefile or sth to make tests easy to run (move the tests imports to correct folder) @farrandi
- Reword this because you do not reference these studies ""Various studies have explored a range of determinants, including age, educational background, language exposure, and the presence of learning disabilities like dyslexia"
- Insertion error in report "Some variable insertion errors: "suggesting a % average prediction error." in method and results section in the report; "RMSE scores and fit times ({numref}lasso_top_models-fig)." for figure 3."
- Figure 4 is too small compare to other figures, and the text looks blur because of low resolution.
- Fix the typos in the report
- Change color scheme of correlation matrix
- Fix the results plot "Regarding Fig. 5, it would be interesting if you discussed more about why there seems to be a straight line acting as a boundary on the upper right-hand side of your scatterplot. Also, the graph seem a little bit crowded, it would probably be better if you made it a little bit bigger and if you made the dots more translucent or changed the color scheme to one that allow us to see the density (how crowded) of different sections of the plot (maybe a 2-D histogram would be better)."
rbouwer