LR-POR / MorphoBr

Resources for morphological analysis of Portuguese
Apache License 2.0
24 stars 4 forks source link

Organization #33

Closed arademaker closed 6 years ago

arademaker commented 6 years ago

@leoalenc, @odanoburu e @fcbr

sei que é muito chato estas constantes reorganizações de nomes de arquivos e diretórios, mas é natural continuarmos tentando achar um balanceamento que seja mais fácil de trabalhar e atenda demandas. Meu racional aqui:

  1. @leoalenc pede que as classes sejam mantidas separadas. Estou preservando isso nos prefixos.

  2. @odanoburu diz que se preocupa com diff dos arquivos no repo. Acho que isto pode ser interessante mas na prática divisões muito manuais só dificultam nossa vida. O diff é eficiente e fácil de usar.

  3. @fcbr teve dificuldade de entender os arquivos quando começou a colaborar, boa pista de que estava confuso.

O que acham? Se concordarem basta aceitar o PR.

fcbr commented 6 years ago

Indo pela sua descricao parece bom (o github esta com dificuldades em mostrar o commit). So peco que inclua tambem um script que gera o dicionario compilado tambem.

arademaker commented 6 years ago

Neste caso ficaria trivial, todos os arquivos dentro do dict e seus subdiretórios. Mas sim, posso fazer também.

odanoburu commented 6 years ago

image

acho que quebramos alguma coisa, hahaha

eu não tenho me preocupado mt com os diffs agora que concordamos que vamos fazer releases!

vou fazer um checkout local pra ver como ficou só

arademaker commented 6 years ago

de qq modo, não vamos aceitar este PR, já estamos mexendo no master, este branch é apenas para mostrar como poderia ficar. Assim que concodarmos, depois de resolver alguns issues pendentes rearrumamos de uma vez.

odanoburu commented 6 years ago

pq não colocar a extensão comum? da forma que está há um README no meio dos arquivos do dicionário..

arademaker commented 6 years ago

Por que com o split não dá para gerar extensão, uma pena. Mas sim, isto é um problema. Mas uma razão para pensarmos um pouco mais... talvez @fcbr tenha razão e vc com sua idéia de release, basta termos um script de 'deploy'

odanoburu commented 6 years ago

@arademaker na minha versão dá!

       --additional-suffix=SUFFIX
              append an additional SUFFIX to file names
arademaker commented 6 years ago

macos versus gnu utilities.

odanoburu commented 6 years ago

nesse caso, posso fazer uma contraproposta/PR?

mantendo os diretórios e colocando as extensões?

odanoburu commented 6 years ago

macos versus gnu utilities.

yep, e GNU > macOS :stuck_out_tongue: