LR-POR / tools

Tools for checking the compatibility between a lexical resource and a treebank
BSD 3-Clause "New" or "Revised" License
2 stars 0 forks source link

Correções nos arquivos de clíticos do MorphoBr #29

Closed analununes closed 2 years ago

analununes commented 2 years ago

O objetivo dessa issue é corrigir os erros das issues 48 e 81 do MorphoBr.

A issue 48 será resolvlda adicionando a tag V quando ela não estiver presente na entrada. Para corrigir a questão da issue 81 a tag PTPASS será trocada pela tag PTPST, que ainda não foi incluída nesse repositório.

analununes commented 2 years ago

Como o diretório de clíticos não está organizado alfabeticamente como os outros diretórios do MorphoBr - e não parece ser uma boa ideia usar essa organização devido ao tamanho dos arquivos que seriam gerados - implementei no commit 09113d1bf904327ed16536da7641b42183263e6b a função delclitics em vez de usar a função delete, que depende da organização em ordem alfabética.

A função delclitics, funciona de forma simples, para cada entrada de cada arquivo, se a entrada existe na lista de entradas a serem apagadas, a mesma é eliminada.

arademaker commented 2 years ago

e não parece ser uma boa ideia usar essa organização devido ao tamanho dos arquivos que seriam gerados

Não está claro para mim esta parte, pode esclarecer. Em comentário anterior, sugeri uma organização da biblioteca en camadas, o que resultaria em uma única função para adição e outra para remoção de entradas.

analununes commented 2 years ago

e não parece ser uma boa ideia usar essa organização devido ao tamanho dos arquivos que seriam gerados

Não está claro para mim esta parte, pode esclarecer. Em comentário anterior, sugeri uma organização da biblioteca en camadas, o que resultaria em uma única função para adição e outra para remoção de entradas.

Estava preocupada com o tamanho dos arquivos que seriam gerados mas agora vejo que realmente faz sentido padronizar os diretórios e utilizar funções únicas. Abri a issue #31 para reorganizar os arquivos de clíticos.