Skip to content

Commit

Permalink
Since we now have ExtPos, no need to skip rel-upos-det for fixed expr…
Browse files Browse the repository at this point in the history
  • Loading branch information
dan-zeman committed Nov 18, 2024
1 parent 6fb65d2 commit 715fcc8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion validate.py
Original file line number Diff line number Diff line change
Expand Up @@ -1477,7 +1477,7 @@ def validate_upos_vs_deprel(id, tree):
#if re.match(r"^(nsubj|obj|iobj|obl|vocative|expl|dislocated|nmod|appos)", deprel) and re.match(r"^(VERB|AUX|ADV|SCONJ|CCONJ)", upos):
# warn("Node %s: '%s' should be a nominal but it is '%s'" % (cols[ID], deprel, upos), 'Syntax', lineno=-1)
# Determiner can alternate with a pronoun.
if deprel == 'det' and not re.match(r"^(DET|PRON)", upos) and not 'fixed' in childrels:
if deprel == 'det' and not re.match(r"^(DET|PRON)", upos):
testid = 'rel-upos-det'
testmessage = "'det' should be 'DET' or 'PRON' but it is '%s'" % (upos)
warn(testmessage, testclass, testlevel, testid, nodeid=id, lineno=tree['linenos'][id])
Expand Down

0 comments on commit 715fcc8

Please sign in to comment.