search

UniversalDependencies / UD_Portuguese-Bosque

This Universal Dependencies (UD) Portuguese treebank.
Other
48 stars 11 forks source link

Issue 300 #321

Closed wellington36 closed 3 years ago

arademaker commented 3 years ago

@wellington36 , veja que comentei os recorrentes casos de AO verbo. E acabei de lembrar que @leoalenc fez um bom comentário sobre esta construção no https://github.com/UniversalDependencies/UD_Portuguese-Bosque/issues/295. Poderia dar uma olhada para depois avaliarmos. acho que vários casos daqui poderão ser analisados como sugerido pelo @leoalenc

wellington36 commented 3 years ago

Veja o commit f7ee7f53966b47ce36b0e2c2f40f95d48d98b3a1, pelo que entendi do issue #295 é isso que queremos, fiz essa mudança baseado no issue 295, nos outros casos que temos de SCONJ (não encontrei nenhum caso de "ao" como SCONJ faz sentido dado que estávamos quebrando em 2 tokens sempre) e no próprio conllu. Acredito que faz sentido a modificação e queremos expandir para os outros casos.

leoalenc commented 3 years ago

Revi 24 dos 71 casos. Peço aplicar as correções nos comentários que fiz @wellington36. Temos vários casos de pelo menos... e ao VERB... sugiro uma análise mais consistente e, se possível em batch dos casos. Talvez @leoalenc possa nos ajudar em confirmar as analises destas construções.

@arademaker, pode transcrever aqui alguns exemplos?

arademaker commented 3 years ago

@leoalenc vc quer exemplos? Vide https://github.com/UniversalDependencies/UD_Portuguese-Bosque/issues/323 para os casos de pelo menos

arademaker commented 3 years ago

Para os casos de ao VERB vide issue https://github.com/UniversalDependencies/UD_Portuguese-Bosque/issues/295

wellington36 commented 3 years ago

@arademaker verifiquei que todos os casos de ao menos e pelo menos tratados aqui são cobertos pelo issue #323 e todos os casos de ao VERB são cobertos pelo issue #295.

arademaker commented 3 years ago

@wellington36 melhor sempre responder comentários depois que novo commit for adicionado no PR.

arademaker commented 3 years ago

@wellington36 vc já atendeu aos meus últimos pedidos neste PR? Posso fazer o merge?

wellington36 commented 3 years ago

Sim, o commit que atendeu os últimos pedidos foi o a29972e7790bf19307011777c2f59c757f4b9b01.