search

AY1920S1-CS2103-F10-3 / main

CS2103/T Revision Tool
https://ay1920s1-cs2103-f10-3.github.io/main/
MIT License
0 stars 4 forks source link

Urgent: JAR file too big #182

Open damithc opened 4 years ago

damithc commented 4 years ago

Guys, your JAR file is too big. It's going to cause problems in the PE. How did it become so big?

damithc commented 4 years ago

It's OK if you can't make it smaller. It's my fault that I did not impose a max file size for the jar file :-p

jxofficial commented 4 years ago

Dear Professor Damith,

The large file size is due to the Stanford NLP library we imported. We have consulted the member in charge of the input validation feature, and he says that he has tried his best to reduce the file size its minimum, which is still 380+mb.

The PE dry run seemed to run smoothly for our evaluators, and hopefully that will be still the case for the actual PE.

Thank you for flagging this issue, and do let us know if there is anything else we need to do to ensure that the PE can proceed without problems.

damithc commented 4 years ago

The size is a problem when we distribute PE files. We distribute PE files as encrypted zip files. When one file is big, we have to make every zip file big or else the file size will reveal the target product, which we don't want to reveal in advance. You see the problem? :-p Imagine asking the whole class to download a 400Mb file. Anyway, no worries. We'll figure something out.