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.

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

Peer Review - Frontend data & Functional Patterns #1

Open
veerleprins opened this issue Nov 12, 2020 · 0 comments
Open

Peer Review - Frontend data & Functional Patterns #1

veerleprins opened this issue Nov 12, 2020 · 0 comments

Comments

@veerleprins
Copy link

Scan someone else's repository

  • Repo is correctly named (frontend data) ✅
  • There is a .gitignore with the right names in it. ✅
  • There is a link to github pages, but nothing of a static / interactive graph can be seen here.
  • There is a description, just not yet exactly what you do specifically in this repo. (by this I mean: an interactive visualization with the library D3 in javascript to set the park and rides against the years, for example)
  • It is clear what your concept is! Perhaps you can add images to your wiki to be able to see what you are doing in 1 view (without having to read the entire text), this helps the reader. ✅
  • No image of the visualization has yet been displayed in the readme.
  • No example has yet been named in the readme from which you got your code.
  • You do have your main question, which data you want to use and a sketch in your wiki to which you link from your ReadMe. ✅
  • You have not yet described how you use your data from the RDW (neither in the wiki nor in your readme). The wiki only contains information about the survey data!
  • You also haven't really described where you got your code from (from curran, among others, I assume) and almost no source reference. 😔

Wiki

  • I can follow the data cleaning and transformation process, but you have nothing based on your RDW data (only on your survey data). Not that that's bad for functional programming, but as a reader I now have no idea how or whether you cleaned up your RDW data.
  • The update pattern has not yet been documented.
  • You could therefore still describe in your wiki what you have done with your RDW data and what you have learned from it, this is now unknown to me!

Live demo

  • There is no live demo available yet. 😔
  • I can see your graph in the vizhub, that looks good 👍

Note:

Nice that you made a pie chart!
Try to go through every point of the rebric today and see: do I have everything?
But that will be fine, good luck 💪

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

No branches or pull requests

1 participant