tP: v2.0
Guidance for the item(s) below:
This week, you get a chance to fix your tP bugs (in the project, as well as documentation) without any penalty. What's more, others will help you find those bugs (via tutorial activities and the PE Dry Run happening in this week).
To take advantage of the above, try to make your v2.0 (product, DG, and UG) as close to what you intend to submit as your final version (i.e., v2.1).
Coming soon
.Admin tP → Deliverables → User Guide
In UG/DG, using hierarchical section numbering and figure numbering is optional (reason: it's not easy to do in Markdown), but make sure it does not inconvenience the reader (e.g., use section/figure title and/or hyperlinks to point to the section/figure being referred to). Examples:
In the section Implementation given above ...
CS2113 does not require you to indicate author name of DG/UG sections (CS2101 requirements may differ). We recommend (but not require) you to ensure that the code dashboard reflect the authorship of doc files accurately.
The main content you add should be in the docs/UserGuide.md
file (for ease of tracking by grading scripts).
Should cover all v2.1 features.
Ensure those descriptions match the product precisely, as it will be used by peer testers (inaccuracies will be considered bugs).
Optionally, can also cover future features. Mark those as Coming soon
.
It is not necessary for the UG to contain every nitty-gritty detail about the product behavior. Some rarely needed information can be omitted from the UG, if the user is expected to know that information already or if the user is kept informed in other ways. For example, if a certain invalid input is unlikely to be used anyway, it is fine to not specify it in the UG, as long as the product is able to give an informative error message when that invalid input is used.
Beware of overusing screenshots. While it is good to have screenshots in the UG, note that they are hard to maintain. For example, if a future version changes the GUI slightly, it will require all your screenshots to be updated. Here are some tips:
Also note the following constraint:
Admin tP Contstraints → Constraint-File-Size
The file sizes of the deliverables should not exceed the limits given below.
Reason: It is hard to download big files during the practical exam due to limited WiFi bandwidth at the venue:
Product (i.e., the JAR/ZIP file): 100MB (Some third-party software -- e.g., Stanford NLP library, certain graphics libraries -- can cause you to exceed this limit)
Documents (i.e., PDF files): 15MB/file (Not following the recommended method of converting to PDF format can cause big PDF files. Another cause is using unnecessarily high resolution images for screenshots).
Admin tP Contstraints → Constraint-Java-Version
Allocation for DG review
Your GitHub | First choice | Second choice | Third choice |
---|---|---|---|
CabbageTime |
CS2113T-W09-2 DG PR |
CS2113-T10-2 DG PR |
CS2113T-W09-3 DG PR |
heyjinwei |
CS2113T-W09-2 DG PR |
CS2113-T10-2 DG PR |
CS2113T-W09-3 DG PR |
jadenwjh |
CS2113T-W09-3 DG PR |
CS2113-T10-3 DG PR |
CS2113T-W09-4 DG PR |
warmwhalefy |
CS2113T-W09-3 DG PR |
CS2113-T10-3 DG PR |
CS2113T-W09-4 DG PR |
song0180 |
CS2113T-W09-3 DG PR |
CS2113-T10-3 DG PR |
CS2113T-W09-4 DG PR |
lowwilliam |
CS2113T-W09-3 DG PR |
CS2113-T10-3 DG PR |
CS2113T-W09-4 DG PR |
baggiiiie |
CS2113T-W09-3 DG PR |
CS2113-T10-3 DG PR |
CS2113T-W09-4 DG PR |
PingruiLi |
CS2113T-W09-4 DG PR |
CS2113-T10-4 DG PR |
CS2113T-W09-3 DG PR |
beaniestanley |
CS2113T-W09-4 DG PR |
CS2113-T10-4 DG PR |
CS2113-F10-1 DG PR |
lihaoyangML |
CS2113T-W09-4 DG PR |
CS2113-T10-4 DG PR |
CS2113-F10-1 DG PR |
cloudy3 |
CS2113T-W09-4 DG PR |
CS2113-T10-4 DG PR |
CS2113-F10-1 DG PR |
justinaquak |
CS2113T-W09-4 DG PR |
CS2113-T10-4 DG PR |
CS2113-F10-1 DG PR |
joohwan58 |
CS2113-F10-1 DG PR |
CS2113-W10-1 DG PR |
CS2113-F10-2 DG PR |
SimJJ96 |
CS2113-F10-1 DG PR |
CS2113-W10-1 DG PR |
CS2113-F10-2 DG PR |
leeyp |
CS2113-F10-1 DG PR |
CS2113-W10-1 DG PR |
CS2113-F10-2 DG PR |
kwokyto |
CS2113-F10-1 DG PR |
CS2113-W10-1 DG PR |
CS2113-F10-2 DG PR |
Vinci-Hu |
CS2113-F10-2 DG PR |
CS2113-W10-1 DG PR |
CS2113-F10-3 DG PR |
tehtea |
CS2113-F10-2 DG PR |
CS2113-W10-2 DG PR |
CS2113-F10-3 DG PR |
chenling1022 |
CS2113-F10-2 DG PR |
CS2113-W10-2 DG PR |
CS2113-F10-3 DG PR |
geezzzyyy |
CS2113-F10-2 DG PR |
CS2113-W10-2 DG PR |
CS2113-F10-3 DG PR |
zhangyongzhe20 |
CS2113-F10-3 DG PR |
CS2113-W10-2 DG PR |
CS2113-T10-1 DG PR |
cswbibibi |
CS2113-F10-3 DG PR |
CS2113-W10-3 DG PR |
CS2113-T10-1 DG PR |
violinyap |
CS2113-F10-3 DG PR |
CS2113-W10-3 DG PR |
CS2113-T10-1 DG PR |
L-Irvin |
CS2113-F10-3 DG PR |
CS2113-W10-3 DG PR |
CS2113-T10-1 DG PR |
Zufiqqar |
CS2113-T10-1 DG PR |
CS2113-W10-3 DG PR |
CS2113-T10-2 DG PR |
boonjuey |
CS2113-T10-1 DG PR |
CS2113-W10-3 DG PR |
CS2113-T10-2 DG PR |
Tyuanyuan |
CS2113-T10-1 DG PR |
CS2113T-F08-1 DG PR |
CS2113-T10-2 DG PR |
JoviYeung92 |
CS2113-T10-1 DG PR |
CS2113T-F08-1 DG PR |
CS2113-T10-2 DG PR |
nagiteja |
CS2113-T10-1 DG PR |
CS2113T-F08-1 DG PR |
CS2113-T10-2 DG PR |
KevinNgWK |
CS2113-T10-2 DG PR |
CS2113T-F08-1 DG PR |
CS2113-T10-1 DG PR |
brynagoh |
CS2113-T10-2 DG PR |
CS2113T-F08-1 DG PR |
CS2113-T10-3 DG PR |
kangxinwang |
CS2113-T10-2 DG PR |
CS2113T-F08-2 DG PR |
CS2113-T10-3 DG PR |
JethroPhuah |
CS2113-T10-2 DG PR |
CS2113T-F08-2 DG PR |
CS2113-T10-3 DG PR |
ManikaHennedige |
CS2113-T10-2 DG PR |
CS2113T-F08-2 DG PR |
CS2113-T10-3 DG PR |
averliz |
CS2113-T10-3 DG PR |
CS2113T-F08-2 DG PR |
CS2113-T10-4 DG PR |
lamzf1998 |
CS2113-T10-3 DG PR |
CS2113T-F08-2 DG PR |
CS2113-T10-4 DG PR |
e0699194 |
CS2113-T10-3 DG PR |
CS2113T-F08-3 DG PR |
CS2113-T10-4 DG PR |
FarmZH98 |
CS2113-T10-3 DG PR |
CS2113T-F08-3 DG PR |
CS2113-T10-4 DG PR |
jalvinchan |
CS2113-T10-3 DG PR |
CS2113T-F08-3 DG PR |
CS2113-T10-4 DG PR |
yanli1215 |
CS2113-T10-4 DG PR |
CS2113T-F08-3 DG PR |
CS2113-T10-3 DG PR |
Chihui8199 |
CS2113-T10-4 DG PR |
CS2113T-F08-4 DG PR |
CS2113-W10-1 DG PR |
jovanhuang |
CS2113-T10-4 DG PR |
CS2113T-F08-4 DG PR |
CS2113-W10-1 DG PR |
NgManSing |
CS2113-T10-4 DG PR |
CS2113T-F08-4 DG PR |
CS2113-W10-1 DG PR |
s-t-e-f |
CS2113-T10-4 DG PR |
CS2113T-F08-4 DG PR |
CS2113-W10-1 DG PR |
yyixue |
CS2113-W10-1 DG PR |
CS2113T-T09-1 DG PR |
CS2113T-F08-1 DG PR |
seangoats |
CS2113-W10-1 DG PR |
CS2113T-T09-1 DG PR |
CS2113-W10-2 DG PR |
vaiish371 |
CS2113-W10-1 DG PR |
CS2113T-T09-1 DG PR |
CS2113-W10-2 DG PR |
zihan9485 |
CS2113-W10-1 DG PR |
CS2113T-T09-1 DG PR |
CS2113-W10-2 DG PR |
vvvvh123 |
CS2113-W10-1 DG PR |
CS2113T-T09-1 DG PR |
CS2113-W10-2 DG PR |
jianningzhuang |
CS2113-W10-2 DG PR |
CS2113T-T09-2 DG PR |
CS2113-W10-3 DG PR |
bryanwhl |
CS2113-W10-2 DG PR |
CS2113T-T09-2 DG PR |
CS2113-W10-3 DG PR |
AlexanderTanJunAn |
CS2113-W10-2 DG PR |
CS2113T-T09-2 DG PR |
CS2113-W10-3 DG PR |
blank-bank |
CS2113-W10-2 DG PR |
CS2113T-T09-2 DG PR |
CS2113-W10-3 DG PR |
NoorSarrah |
CS2113-W10-3 DG PR |
CS2113T-T09-2 DG PR |
CS2113T-T09-1 DG PR |
MingShun98 |
CS2113-W10-3 DG PR |
CS2113T-T09-3 DG PR |
CS2113T-F08-1 DG PR |
Chiamjiaen |
CS2113-W10-3 DG PR |
CS2113T-T09-3 DG PR |
CS2113T-F08-1 DG PR |
huachen24 |
CS2113-W10-3 DG PR |
CS2113T-T09-3 DG PR |
CS2113T-F08-1 DG PR |
jhjhajh |
CS2113-W10-3 DG PR |
CS2113T-T09-3 DG PR |
CS2113T-F08-1 DG PR |
EmilyTJX |
CS2113T-F08-1 DG PR |
CS2113T-T09-4 DG PR |
CS2113-W10-1 DG PR |
Krithigha24 |
CS2113T-F08-1 DG PR |
CS2113T-T09-4 DG PR |
CS2113T-F08-2 DG PR |
BlubberMonster |
CS2113T-F08-1 DG PR |
CS2113T-T09-4 DG PR |
CS2113T-F08-2 DG PR |
hazelhedmine |
CS2113T-F08-1 DG PR |
CS2113T-T09-4 DG PR |
CS2113T-F08-2 DG PR |
nivikcivik |
CS2113T-F08-1 DG PR |
CS2113T-T09-4 DG PR |
CS2113T-F08-2 DG PR |
liping-eng |
CS2113T-F08-2 DG PR |
CS2113T-W09-1 DG PR |
CS2113T-T09-2 DG PR |
JonathanKhooTY |
CS2113T-F08-2 DG PR |
CS2113T-W09-1 DG PR |
CS2113T-F08-3 DG PR |
iamakilahamed |
CS2113T-F08-2 DG PR |
CS2113T-W09-1 DG PR |
CS2113T-F08-3 DG PR |
hussain1998 |
CS2113T-F08-2 DG PR |
CS2113T-W09-1 DG PR |
CS2113T-F08-3 DG PR |
limwenfeng |
CS2113T-F08-2 DG PR |
CS2113T-W09-1 DG PR |
CS2113T-F08-3 DG PR |
sarzorwyn |
CS2113T-F08-3 DG PR |
CS2113T-W09-2 DG PR |
CS2113T-F08-4 DG PR |
wjchoi0712 |
CS2113T-F08-3 DG PR |
CS2113T-W09-2 DG PR |
CS2113T-F08-4 DG PR |
Rye98 |
CS2113T-F08-3 DG PR |
CS2113T-W09-2 DG PR |
CS2113T-F08-4 DG PR |
Rizavur |
CS2113T-F08-3 DG PR |
CS2113T-W09-2 DG PR |
CS2113T-F08-4 DG PR |
SimBowen |
CS2113T-F08-4 DG PR |
CS2113T-W09-2 DG PR |
CS2113T-T09-1 DG PR |
KimIdeas8 |
CS2113T-F08-4 DG PR |
CS2113T-W09-3 DG PR |
CS2113T-T09-1 DG PR |
fangxinjia0203 |
CS2113T-F08-4 DG PR |
CS2113T-W09-3 DG PR |
CS2113T-T09-1 DG PR |
soepaingzaw |
CS2113T-F08-4 DG PR |
CS2113T-W09-3 DG PR |
CS2113T-T09-1 DG PR |
rageqqq |
CS2113T-T09-1 DG PR |
CS2113T-W09-3 DG PR |
CS2113-W10-3 DG PR |
douglaslewpc |
CS2113T-T09-1 DG PR |
CS2113T-W09-3 DG PR |
CS2113T-T09-2 DG PR |
Cocokkkk |
CS2113T-T09-1 DG PR |
CS2113T-W09-4 DG PR |
CS2113T-T09-2 DG PR |
951553394 |
CS2113T-T09-1 DG PR |
CS2113T-W09-4 DG PR |
CS2113T-T09-2 DG PR |
zikunz |
CS2113T-T09-1 DG PR |
CS2113T-W09-4 DG PR |
CS2113T-T09-2 DG PR |
ongweisheng |
CS2113T-T09-2 DG PR |
CS2113T-W09-4 DG PR |
CS2113T-F08-2 DG PR |
E00426142 |
CS2113T-T09-2 DG PR |
CS2113T-W09-4 DG PR |
CS2113T-T09-3 DG PR |
gerardtwk |
CS2113T-T09-2 DG PR |
CS2113-F10-1 DG PR |
CS2113T-T09-3 DG PR |
LeeHanYongAndy |
CS2113T-T09-2 DG PR |
CS2113-F10-1 DG PR |
CS2113T-T09-3 DG PR |
jonahtwl |
CS2113T-T09-2 DG PR |
CS2113-F10-1 DG PR |
CS2113T-T09-4 DG PR |
tzexern |
CS2113T-T09-3 DG PR |
CS2113-F10-1 DG PR |
CS2113T-T09-4 DG PR |
marklowsk |
CS2113T-T09-3 DG PR |
CS2113-F10-2 DG PR |
CS2113T-T09-4 DG PR |
kewenlok |
CS2113T-T09-3 DG PR |
CS2113-F10-2 DG PR |
CS2113T-T09-4 DG PR |
LJ-37 |
CS2113T-T09-3 DG PR |
CS2113-F10-2 DG PR |
CS2113T-T09-4 DG PR |
oscarlai1998 |
CS2113T-T09-4 DG PR |
CS2113-F10-2 DG PR |
CS2113T-T09-3 DG PR |
fupernova |
CS2113T-T09-4 DG PR |
CS2113-F10-3 DG PR |
CS2113T-W09-1 DG PR |
xseh |
CS2113T-T09-4 DG PR |
CS2113-F10-3 DG PR |
CS2113T-W09-1 DG PR |
ivanchongzhien |
CS2113T-T09-4 DG PR |
CS2113-F10-3 DG PR |
CS2113T-W09-1 DG PR |
H-horizon |
CS2113T-T09-4 DG PR |
CS2113-F10-3 DG PR |
CS2113T-W09-1 DG PR |
isaharon |
CS2113T-W09-1 DG PR |
CS2113-T10-1 DG PR |
CS2113T-W09-2 DG PR |
aliciatay-zls |
CS2113T-W09-1 DG PR |
CS2113-T10-1 DG PR |
CS2113T-W09-2 DG PR |
8kdesign |
CS2113T-W09-1 DG PR |
CS2113-T10-1 DG PR |
CS2113T-W09-2 DG PR |
brandonfoong |
CS2113T-W09-1 DG PR |
CS2113-T10-1 DG PR |
CS2113T-W09-2 DG PR |
hiongkaihan |
CS2113T-W09-1 DG PR |
CS2113-T10-1 DG PR |
CS2113T-W09-2 DG PR |
leowxx |
CS2113T-W09-2 DG PR |
CS2113-T10-2 DG PR |
CS2113T-W09-1 DG PR |
fsgmhoward |
CS2113T-W09-2 DG PR |
CS2113-T10-2 DG PR |
CS2113T-W09-3 DG PR |
Emkay16 |
CS2113T-W09-2 DG PR |
CS2113-T10-2 DG PR |
CS2113T-W09-3 DG PR |
Decide which of the given team(s) to review:
Go to the PR of the team(s) you have chosen to review.
Review the Design
and the Implementation
sections w.r.t possible DG bugs (given further down); add your observations as comments.
Is this format correct? Should it be ... instead?
) rather than directives (e.g., Change this to ...
). Pay attention to these as they are same as the final evaluation criteria of the DG.
Admin tP Grading → DG Bugs → UML Diagrams → Possible Bugs
UML diagrams:
Admin tP Grading → General Documentation Bugs
Use of visuals
Use of examples:
Explanations:
Neatness/correctness:
Also see:
Admin tP: Deliverables → DG → Tips
0..1
vs 1
, composition vs aggregation*Command
classes using a placeholder XYZCommand
).ref
frames to break sequence diagrams to multiple diagrams.These class diagrams seem to have lot of member details, which can get outdated pretty quickly:
In this negative example, the text size in the diagram is much bigger than the text size used by the document:
It will look more 'polished' if the two text sizes match.
delete
command