delph-in / matrix

The Grammar Matrix

Home Page:https://matrix.ling.washington.edu/index.html

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Add Error Message Handling for Test by Generation

rosypen opened this issue · comments

Test by Generation makes calls to ACE or LKB which can fail for various reasons (maybe the grammar cant be compiled, maybe no sentences could be parsed/generated, maybe there were memory issues, etc.)

There should be some graceful error message handling on the test by Generation page to report what went wrong and possibly some instructions on what to do. In particular, for test by generation using test sentences, when the test sentences were not able to be parsed it should at least report that an attempt was made but it failed at parsing for example.

The code itself can also lend itself to some bugs as well because there are expectations about the formatting of many things like the choices file or the predication schemes so if any of those change there will likely be an error.

It's a good idea to just put some kind of try-except around potentially problematic areas.