UniversalPetrarch icon indicating copy to clipboard operation
UniversalPetrarch copied to clipboard

Add pattern-based test cases

Open PTB-OEDA opened this issue 7 years ago • 1 comments

Since the coder can now use the PETRARCH 1 dictionaries, test cases need to be added to make sure the patterns work correctly

PTB-OEDA avatar Apr 27 '18 21:04 PTB-OEDA

There are a bunch of these in petrarch/data/dictionaries/PETR.Validate.verbs.txt in the openeventdata/petrarch repo. I'm guessing you'll need to modify them since I assume we're abandoning the underscore notation, but there are a lot of the actor tokens %, +, $ and %. Dig around a bit in the comments in the cases in data/text/PETR.UnitTest.records.xml and it should be possible to match up a lot of the cases and the pattern it matched. But you probably don't want the whole dictionary since it was relatively large and has a bunch of nonsense phrases left over from the TABARI sparse parsing.

philip-schrodt avatar Jun 01 '18 21:06 philip-schrodt