Issues
Tijdens de peer reviews, zijn er bij mij door teamgenoten ook issues aangemaakt.
Voor deze reviews heb ik per onderwerp een issue template geschreven.
See template (click to expand)
about what part of my repo is this issue? e.g. index.html, app.js, readme.md, etc
what is this issue about?
what would you recommend (instead)?
See template (click to expand)
[zet hier je antwoord neer]
[zet hier je antwoord neer]
[zet hier je antwoord neer]
See template (click to expand)
📈 Rubric
Please use the legenda below to assign the values to the criteria (example-filled already)
-
✅: voldoende
-
〽️: aanvulling
-
❓: geen idee
-
❌: nog niet voldoende
[write here what I still need to fix, tips or tops]
- ✅ Git, is used
- ✅ The project runs without errors
- ❓ Data is loaded with d3
- ❓ There is a representation of data
- Data is rendered with d3
- Interpreting the representation is easier that interpreting the data itself
- Functional patterns are used to clean and transform data.
- ❓ Representation and use of d3 go beyond an example.
- ✅ Code is set up in a modular way and has no unexpected side effects
- 😱 The way the student applies subject matter is more advanced than what they were taught in class; let’s switch places
[write here what I still need to fix, tips or tops]
- There is substantial own code
- The student can explain the code that exists
- The student can explain some parts of their code
- The student can explain how some parts works together
- The student can explain some technical choices
- The student can explain every part of their code
- The student can explain how everything works together
- The student can explain why patterns are used instead of alternatives
- The project is structured logically
- The project is complex but can easily be understood
- Alternatives to patterns covered in class was used that were great choices
- 🤓 The student deeply understands functional programming and can create their own functional code flows
[write here what I still need to fix, tips or tops]
- The project is handed in on time
- The project is working
- The project is documented
- The project is on GitHub
- Code style is consistent
- Code and project are partially documented
- Code adheres to standards
- _site cover what the project is and does
- Code quality is good and enforced
- _site are useful and professional
- 📚 Code and _site both read like great books
[write here what I still need to fix, tips or tops]
- Process is partially documented in the wiki
- Process is properly documented
- 〽️ Choices are evaluated and documented
- ✅ Progress is demonstrated
- ❌ Work reflects client's needs
- Significant progress or iterations are demonstrated
- Client is happy with the work
- 💪 What you did this course is amazing; Teachers and client are in awe of your progress
Any more comments? Drop them below 👇🏼
Peer Review 1: Zekkie
"Zijn er aannames die je hebt die je wellicht kan tegen komen in de datasets die jij gaat gebruiken?"
Ik heb toen mijn aannames toegevoegd aan de readme.
Peer Review 2: Zekkie
Ja!
- Hoe ver liggen de corona teststraten van elkaar af.
- Wat is de gemiddelde capaciteit van een teststraat.
- Van hoelaat tot hoelaat is een teststraat open.
Ik wachtte nog even tot ik wist of er überhaupt een dataset met de locaties van de coronateststraten beschikbaar is, anders kon ik de vragen die daarmee te maken hebben misschien beter laten vallen
Verder heb ik de dataset-suggestie toegevoegd aan de lijst met resources.
Peer Review 3: Zekkie
Code examples
Could you link to code examples?
Ik heb alle code examples toegevoegd aan m'n wiki.
Peer Review 4: Zekkie
clean-eyecolor.js
Async/await
since .map() doesn't return a Promise, you shouldn't be using [await] (https://github.com/deannabosschert/functional-programming/blob/a9b74c4d47eb8cc1e7f5b14e711238847b3c6733/src/lib/clean-eyecolor.js#L6) here
Ik heb await weggehaald.
Peer Review 5: Zekkie
clean-eyecolor.js
functional-programming/src/lib/clean-eyecolor.js
Line 18 in a9b74c4
return eyeColors = questionnaire.map(person => ({You could write the the code like
function getEyeColors(questionnaire) {
return questionnaire.map(person => ({
eyeColor: person.oogKleur
}))
}Ik heb het advies opgevolgd.
Peer Review 6: Zekkie
Wiki
No JSON formatting example
You've described the contents of a datum, but you could also add an example of a datum from the survey data in JSON format.
Ik heb wat voorbeelden toegevoegd aan de wiki-pagina over data cleaning.
Peer Review 7: Zekkie
Welke datapunten denk je nodig te gaan hebben uit de datasets?
Ik heb wat voorbeelden toegevoegd.
Peer Review 8: Jelmer
Zeker! Duidelijk geformuleerd.👍
-
Zijn er bepaalde plekken/parkeer plaatsen waar het heel druk is geworden door het coronavirus? (bijv. bij supermarkten omdat mensen als een gek gingen hamsteren)
🅿️ -
Zijn er parkeerplekken die nu belemmerd worden omdat er een coronateststraat op staat?⛔
-
Hoeveel effort doen parkeerbeheerders om ervoor te zorgen dat bepaalde maatregelen worden nageleefd?😰
Geen data, maar misschien wel handige bronnen
- https://www.verkeersnet.nl/parkeren/35060/extra-coronaregels-leiden-niet-tot-minder-parkeertransacties/
- https://www.verkeersnet.nl/parkeren/35241/parkeren-in-coronatijd-zo-regel-je-het-goed-als-beheerder/
Ik heb de bronnen bekeken en m'n waardering vergeten uit te spreken; ik heb de subvragen meegenomen in mijn concept!