delph-in / matrix

The Grammar Matrix
https://matrix.ling.washington.edu/index.html
Other
12 stars 6 forks source link

Checking for length of vars before modulo #742

Closed ajcassell closed 1 month ago

ajcassell commented 1 month ago

412 I was able to resolve this problem by adding length safeguard checks before the modulus operation. This allows for test by generation to work properly with choices files that either do not have transitive verbs or do not have intransitive verbs (which was the original issue). However, ace outputs an error file (below) indicating that that type of verb is missing. Is this alright?

Screen Shot 2024-09-30 at 5 12 34 PM

Screen Shot 2024-09-30 at 5 16 02 PM
emilymbender commented 1 month ago

Yes, this looks fine.