Skip to content

added UCxn annotation#551

Merged
nschneid merged 6 commits into
UniversalDependencies:devfrom
LeonieWeissweiler:dev
Oct 31, 2024
Merged

added UCxn annotation#551
nschneid merged 6 commits into
UniversalDependencies:devfrom
LeonieWeissweiler:dev

Conversation

@LeonieWeissweiler

Copy link
Copy Markdown
Contributor

Adds construction annotation in the MISC column, as described in https://aclanthology.org/2024.lrec-main.1471.pdf

Annotates NPN, Interrogatives, Conditionals, and Existentials, on the head of the respective phrase, plus construction elements.

For full specification and a subset of the treebank with only cxn-annotated sentences see here: https://github.com/LeonieWeissweiler/UCxn

@nschneid

Copy link
Copy Markdown
Contributor

Thanks! One problem: this is overriding existing Cxn annotations (for relative clauses, which start with rc-). Is it possible to just add Cxn entries without removing any?

@LeonieWeissweiler

Copy link
Copy Markdown
Contributor Author

That's the way that @s-herrera 's script currently works, I'm not sure if it needs to stay that way or could be changed

Comment thread en_ewt-ud-dev.conllu
4 Bay Bay PROPN NNP Number=Sing 5 compound 5:compound _
5 View View PROPN NNP Number=Sing 3 obj 3:obj _
6 if if SCONJ IN _ 9 mark 9:mark _
7 you you PRON PRP Case=Nom|Person=2|PronType=Prs 9 nsubj 9:nsubj _
8 are be AUX VBP Mood=Ind|Number=Sing|Person=2|Tense=Pres|VerbForm=Fin 9 aux 9:aux _
9 looking look VERB VBG Tense=Pres|VerbForm=Part 3 advcl 3:advcl:if _
9 looking look VERB VBG Tense=Pres|VerbForm=Part 3 advcl 3:advcl:if CxnElt=3:Conditional-UnspecifiedEpistemic-NoInversion.Protasis

@nschneid nschneid Oct 26, 2024

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happened to notice that this one is a speech act conditional. I don't think we have a specific label for those.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happened to notice that this one is a speech act conditional. I don't think we have a specific label for those.

Hi, @nschneid !
I guess it wouldn't be right to tag «if» in features as a Protasis marker, but it would be helpful for distinguishing the «conditional» marking in Mordvin languages, where the verbs take a protasis morpheme and called «Conditional» in traditional grammars.
Would you have any suggestions for a «Protasis» feature?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rueter The UCxn approach implemented here is to identify the content elements of the construction via Grew rules that incorporate grammatical marking. We don't tag the functional morphemes like "if" directly. UD morphological features (at least as I understand them) are generally for indicating where a word form lies in a paradigm, not for what an uninflecting function word like "if" contributes to an analytic construction. Does that answer your question?

@s-herrera

Copy link
Copy Markdown

Thanks! One problem: this is overriding existing Cxn annotations (for relative clauses, which start with rc-). Is it possible to just add Cxn entries without removing any?

Yes, it removes all the existing Cxn annotations. It can be changed, but it means that two different annotations for the Cxn field will coexist in the same treebank.

@nschneid

nschneid commented Oct 26, 2024

Copy link
Copy Markdown
Contributor

I think we need to be able to support Cxn annotations that are not from the Grew rules, because many treebank maintainers will want to add language-specific constructions by other means (other automatic means, manual annotation, or a combination thereof).

@s-herrera

Copy link
Copy Markdown

Yes, I agree. I was just saying that, in this particular case, two naming convention will coexist in the same treebank. I have sent the new treebanks to @LeonieWeissweiler so she can update the PR.

@LeonieWeissweiler

Copy link
Copy Markdown
Contributor Author

PR has been updated!

Comment thread en_ewt-ud-dev.conllu Outdated
Comment thread en_ewt-ud-dev.conllu
11 how how ADV WRB PronType=Int 12 advmod 12:advmod _
12 fanatic fanatic ADJ JJ Degree=Pos 9 ccomp 9:ccomp _
11 how how ADV WRB PronType=Int 12 advmod 12:advmod CxnElt=12:Interrogative-WHInfo-Indirect.WHWord
12 fanatic fanatic ADJ JJ Degree=Pos 9 ccomp 9:ccomp Cxn=Interrogative-WHInfo-Indirect|CxnElt=12:Interrogative-WHInfo-Indirect.Clause

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is actually exclamative not interrogative (but it would require manual disambiguation).

Comment thread en_ewt-ud-dev.conllu
12 they they PRON PRP Case=Nom|Number=Plur|Person=3|PronType=Prs 13 nsubj 13:nsubj _
13 call call VERB VBP Mood=Ind|Number=Plur|Person=3|Tense=Pres|VerbForm=Fin 10 ccomp 10:ccomp _
13 call call VERB VBP Mood=Ind|Number=Plur|Person=3|Tense=Pres|VerbForm=Fin 10 ccomp 10:ccomp Cxn=Interrogative-WHInfo-Indirect|CxnElt=13:Interrogative-WHInfo-Indirect.Clause

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tree structure is wrong—should actually be a free relative, #278 (there are a lot of these errors unfortunately, just noting for later)

Comment thread en_ewt-ud-dev.conllu Outdated
Comment thread en_ewt-ud-dev.conllu
@nschneid

Copy link
Copy Markdown
Contributor

Did some spot checks. @LeonieWeissweiler @s-herrera would you mind fixing the wordform=__EMPTY__ issue?

Two limitations that I noticed for documenting in the README:

  • Some free relatives are incorrectly analyzed as interrogative clauses (in the tree and UCxn annotations)
  • Polar interrogative existentials ("Is there...") are not marked as interrogative

@LeonieWeissweiler

Copy link
Copy Markdown
Contributor Author

the wordform empty issue has been fixed, thanks for pointing that out!

@nschneid

Copy link
Copy Markdown
Contributor

Wes has added a rule to fix the polar interrogative existential issue. Sorry to bug you but would you mind running the rules again?

@LeonieWeissweiler

Copy link
Copy Markdown
Contributor Author

fixed now, all credit to @s-herrera who keeps running them :)

@nschneid

Copy link
Copy Markdown
Contributor

Looking at the last commit, something went wrong...maybe it was based on an earlier version of the data?

@s-herrera

Copy link
Copy Markdown

Yes, sorry about that... I fixed it and sent, hopefully, the corrected files to @LeonieWeissweiler.

@nschneid

Copy link
Copy Markdown
Contributor

Great. I undid the spurious commit.

@LeonieWeissweiler

Copy link
Copy Markdown
Contributor Author

does this one work?

@nschneid

Copy link
Copy Markdown
Contributor

I suspect the files are correct, but so we can see the changes, the new commit should be based on the "removed wordform empty" commit not the buggy "fixes to UCxn polarity questions" commit. I will attempt a rebase.

Comment thread en_ewt-ud-dev.conllu
4 not not PART RB Polarity=Neg 5 advmod 5:advmod _
5 available available ADJ JJ Degree=Pos 6 advcl 6:advcl:if CxnElt=6:Conditional-UnspecifiedEpistemic-NoInversion.Protasis
6 is be VERB VBZ Mood=Ind|Number=Sing|Person=3|Tense=Pres|VerbForm=Fin 0 root 0:root Cxn=Conditional-UnspecifiedEpistemic-NoInversion,Existential-CopPred-ThereExpl|CxnElt=6:Conditional-UnspecifiedEpistemic-NoInversion.Apodosis
6 is be VERB VBZ Mood=Ind|Number=Sing|Person=3|Tense=Pres|VerbForm=Fin 0 root 0:root Cxn=Conditional-UnspecifiedEpistemic-NoInversion,Existential-CopPred-ThereExpl,Interrogative-Polar-Direct|CxnElt=6:Conditional-UnspecifiedEpistemic-NoInversion.Apodosis,6:Interrogative-Polar-Direct.Clause

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good example of a sentence with 3 different kinds of constructions

@nschneid

Copy link
Copy Markdown
Contributor

I learned a new trick: git reset --soft HEAD~2 squashes changes from the previous two commits together for a new commit. Looks good now. :)

@amir-zeldes

Copy link
Copy Markdown
Contributor

it would be helpful for distinguishing the «conditional» marking in Mordvin languages, where the verbs take a protasis orpheme and called «Conditional» in traditional grammars. Would you have any suggestions for a «Protasis» feature?

@rueter if you're looking to annotate words like "if" in conditionals, that sounds a bit like discourse relation + discourse marker annotation (in eRST) or connective annotation (in PDTB) - UD_English-GUM annotates those in both frameworks, so you can take a look there/let me know if you want more information on either. This includes disambiguating the relation signaled by e.g. "if", which is not always a conditional (for example: "there's water in the fridge if you want some").

@nschneid nschneid merged commit d1338d5 into UniversalDependencies:dev Oct 31, 2024
@nschneid

Copy link
Copy Markdown
Contributor

Merged with a README note, thanks all!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants