Skip to content

Commit b45657a

Browse files
D2 & D3 Feedback
1 parent 8c354e0 commit b45657a

File tree

2 files changed

+115
-0
lines changed

2 files changed

+115
-0
lines changed
Lines changed: 84 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,84 @@
1+
Vector Institute (ML Simulation) - Team 22
2+
3+
PLAN.MD - Overall Grade - 2
4+
5+
Roles & Responsibilities: 3
6+
- Meets expectations
7+
8+
Communication: 2
9+
- Your process of communicating with the partner could be more detailed (i.e. who is responsible for reaching out, providing updates, setting up meetings)
10+
11+
Meetings: 2
12+
- Is there a moderator for your meetings?
13+
14+
Conflict Resolution: 3
15+
- Meets expectations
16+
17+
Events: 3
18+
- Meets expectations
19+
20+
Partner Meetings: 3
21+
- meets expectations
22+
23+
Deployment and Github: 2
24+
- Is there a more specific naming convention for your branches?
25+
- Lots of typos and missing a lot of periods
26+
- You should still have described the process of creating a docker image in more detail - even if the partner does not want a deployed app
27+
28+
Product
29+
30+
Goals & Tasks: 3
31+
- Is this organised from most to least important?
32+
- Meets expectations
33+
34+
Artifacts: 3
35+
- Very detailed!
36+
37+
Highlights
38+
- You don't need meeting highlights here - this should go in your review documents
39+
40+
--------------------------------------------------------------------------------------------------------------------------------------------------------
41+
42+
REVIEW.MD - Overall Grade -
43+
44+
Process - Reflection
45+
46+
Good Decisions: 3
47+
- meets expectations, thorough
48+
49+
Bad Decisions: 3
50+
- You mentioned that you did have brain storming sessions before coding, I assume that you did not discuss software architecture then?
51+
52+
Planned Changes: 3
53+
- meets expectations
54+
55+
Product
56+
57+
Goals Met: 3
58+
- Meets expectations
59+
60+
Goals Not Met: 3
61+
- meets expectations
62+
63+
Demo: 4
64+
- Very thorough!
65+
66+
Highlights: 3
67+
- Meets expectations
68+
69+
-----------------------------------------------------------------------------
70+
README.md
71+
72+
Description: 3
73+
- Very clear and detailed
74+
75+
Key Features: 3
76+
- meets expectations
77+
78+
Instructions: 4
79+
- Very clear
80+
81+
Deployed App: 3
82+
- meets expectations
83+
84+
Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
Vector Institute
2+
3+
FINAL.md
4+
5+
Changes: 100
6+
- That's great to hear that you didn't make any significant changes, great work!
7+
- Great to see highlights for this deliverable even though it was not required!
8+
9+
Handoff Plan: 85
10+
- Very detailed
11+
12+
README.md
13+
- Very well organized!
14+
15+
Description: 85
16+
- Very detailed!
17+
18+
Key Features: 75
19+
- Great that you have screenshots, wish there was a bit more explanation of your features though
20+
21+
Instructions: 85
22+
- Very thorough
23+
24+
Dev. Requirements: 100
25+
- Very thorough and detailed!
26+
27+
Licensing: 85
28+
- Meets expectations
29+
30+
Product: 85
31+
- The application looks great! Looks like you have managed to achieve exactly what your partner was looking for

0 commit comments

Comments
 (0)