-
-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Latin verb with mood=gerundive matches whitaker mood=participle #608
Comments
this is fixed in Alpheios Components 3.3.3-qa.20210113579 the code change was to consider a gerundive the same as a verb participle -- @monzug note that for now this change applies only to the treebank disambiguation and not the inflection matching. There is also a corresponding change to the treebank data alpheios-project/alpheios-trees#15 to remove the incorrect tense |
fugiendo (https://texts-test.alpheios.net/text/urn:cts:latinLit:phi0620.phi001.alpheios-text-lat1/passage/1.1) is only verb participle fut. pass. - the present form is gone |
I went a little too far with this change, per #615 It's not actually that Whitaker reports the whole lexeme as being a verb participle when an inflection has the gerundive mood, but rather that the inflection itself has a part of speech which is verb participle but the lexeme itself is a verb. So I need to make this change a little less encompassing and more specific:
|
retested in Alpheios Embedded Library 3.3.1-20200821383 |
another interesting point here, which @rgorman helped clarify -- the treebank reports this as mood=gerundive. The Whitake parser used by Alpheios reports all Gerunds as verb participles (see alpheios-project/morphsvc#11).
I think perhaps we should make a change to the Alpheios treebank adapter to consider a Latin verb with mood=gerundive as being the same as a verb participle. That way we will at least be comparing apples to apples. We'd still have a disconnect here, because the tense in the treebank is present. But we would at least be matching the part of speech.
Originally posted by @balmas in #540 (comment)
The text was updated successfully, but these errors were encountered: