surfacesyntacticud / guidelines

Guidelines for Surface Syntactic Universal Dependencies
https://guidelines.surfacesyntacticud.org/
0 stars 0 forks source link

Move ExtPos in the main features? #31

Closed bguil closed 10 months ago

bguil commented 2 years ago

Apart from the native SUD treebanks, the feature ExtPos was adopted by two UD treebanks: UD_English-EWT and UD_Portuguese-Bosque.

In these two UD corpora, the feature ExtPos is put in the FEATS columns whereas in the SUD native corpora, it is in the MISC column (and this is the same for the UD converted versions of these corpora).

I proposed to follow the UD corpora and to move ExtPos from MISC to FEATS in SUD corpora.

kirianguiller commented 1 year ago

Hello,

I would like to know if it's still a concern ? I like the idea to have the ExtPos as a FEATS, especially if it's what is already done in the two mentioned UD treebanks.

bguil commented 1 year ago

If there is no protest against this, I propose to make the change soon in all SUD native treebanks.

bguil commented 1 year ago

In Grew, ExtPos feature is declared as a FEATS features (in opam library conll, version 1.16.0).

update opam library conll

opam update
opam upgrade conll

Update annotated data

The 4 French SUD native treebansk have been updated.

In order to modify the already other annotated treebanks, you can apply the following GRS (with the last version of the conll library!)

rule r {
  pattern { N [__MISC__ExtPos] }
  commands { N.ExtPos = N.__MISC__ExtPos; del_feat N.__MISC__ExtPos }
}
strat main { Onf (r) }