languagetool-org / portuguese-pos-dict

Portuguese POS tagger
GNU Lesser General Public License v2.1
5 stars 2 forks source link

[pt-PT] Improved both .AFF files for AO45 and AO90 #13

Closed marcoagpinto closed 7 months ago

marcoagpinto commented 7 months ago

Heya, @susanaboatto and @p-goulart

Like I said yesterday, I will do this task bit by bit to make sure I won't screw up.

So, here is an improvement for verb forms of the kind:

ama-a
ama-as
ama-lhe
ama-lhes
ama-me
ama-nos
ama-o
ama-os
ama-se
ama-se-lhe
ama-se-lhes
ama-te
ama-vos

NEW WORDS (AO45): 3.PTPT_45_new_verbs.txt

NEW WORDS (AO90): 6.PTPT_90_new_verbs.txt

PS→ Moments ago I forgot to create a Pull Request, so in the Git software I selected to amend it and repulled it.

p-goulart commented 7 months ago

@marcoagpinto , for the time being, can you work off tagging/clitics instead of main? That's the branch where we're adding the new tokenisation logic. I'll change the base of this PR.

marcoagpinto commented 7 months ago

@p-goulart

Ahhhh… Heya,

How do I work on /clitics?

I have the repository in a folder and I search for 45.aff, and it shows a folder from where I get/commit the .AFFs.

p-goulart commented 7 months ago

Instead of checking out the main branch, checkout tagging/clitics. When creating your new branch, make sure it's based on tagging/clitics.

marcoagpinto commented 7 months ago

Instead of checking out the main branch, checkout tagging/clitics. When creating your new branch, make sure it's based on tagging/clitics.

Ahhhhh… thanks!

I will try to check out that instead of the main.

marcoagpinto commented 7 months ago

@p-goulart @susanaboatto @jaumeortola

HELP!!!!!

Screenshot 2024-01-30 183845

susanaboatto commented 7 months ago

@marcoagpinto you don't need to clone it again if you already have the portuguese-pos-dict repo available. All you need to do is look for the tagging/clitics branch under the repo's branches, as you do with any of your other branches. Try updating the project first if you can't find it 😉

image
marcoagpinto commented 7 months ago

Ahhhh… thanks, @susanaboatto , I will try tonight.

My idea of copying rules from the pt-BR into the pt-PT worked fine, just like I expected.

All I have to do is to change the flags names and numbers.