issues
search
chuckf333
/
DataScienceProject
0
stars
0
forks
source link
issues
Newest
Newest
Most commented
Recently updated
Oldest
Least commented
Least recently updated
Final Peer Review
#9
bwalker20
opened
4 years ago
1
p02 peer review summary
#8
ballen2024
opened
4 years ago
0
Consider adding a brief explanation as to what you mean by "clean."
#7
mnmenator
closed
4 years ago
1
You already mentioned the range of years at the beginning of this writeup, and unless there are years missing (which there aren't), it doesn't seem necessary to show all of the years.
#6
mnmenator
closed
4 years ago
1
Consider using suppressWarnings() to hide the parsing failures from your output.
#5
mnmenator
closed
4 years ago
1
Hanging "(eg. )" at the end of the sentence. Either provide examples of abbreviations or remove this.
#4
mnmenator
closed
4 years ago
1
"It contains 22 variables, an ." Unclear message.
#3
mnmenator
closed
4 years ago
1
Rather than labeling every year, consider labeling in intervals of 25 to 50 years in order to make the labels readable.
#2
mnmenator
closed
4 years ago
0
Consider splitting this up onto multiple lines for better readability.
#1
mnmenator
closed
4 years ago
0