Skip to content

sei-24-jarryl-chia#133

Open
jarryll wants to merge 1 commit into
wdi-sg:masterfrom
jarryll:master
Open

sei-24-jarryl-chia#133
jarryll wants to merge 1 commit into
wdi-sg:masterfrom
jarryll:master

Conversation

@jarryll
Copy link
Copy Markdown

@jarryll jarryll commented Aug 12, 2020

…nality not built in yet.

Deliverable Submission

Please describe your comfort and completeness levels before submitting.

Comfort Level (1-5): 4

Completeness Level (1-5): 4

What did you think of this deliverable?: challenging

How many hours did you spend on this assignment? 4

What part did you spend the most time on? getting the logic right

Does the assignment run without errors? If the assignment has errors, what are they? input doesn't refresh fast enough, so can't capture 2 of the same inputs back to back.

Describe one part of the code or choose one line you would improve upon. What would you improve? more helper functions to keep the main code clean.

What did you think of this deliverable? Any other comments?

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

Successfully merging this pull request may close these issues.

1 participant