A3Data/hermione

UML

Closed this issue · 6 comments

teoria commented

o relacionamento da classe Train com as outras está invertido.
o diamante deveria estar no nado da classe train nas composições

neylsoncrepalde commented

Ei Rodrigo @teoria . Tudo bom?
Obrigado pelo toque. Vamos corrigir.
Abs

barbarasilveiraf commented

Oi Rodrigo, tudo bem?

A nossa classe Train do diagrama representa o script train.py no código, a qual contém ou pode conter as demais classes: DataSource, Preprocessing, Trainer, Visualization, etc.
Por isso o diagrama está correto, pois elas compõe o Train.

Qualquer sugestão estamos a disposição.
Att.,
Bárbara

teoria commented

https://drive.google.com/file/d/1lSXfEUXSt-BSczA02AFLmzaM140S3UXd/view?usp=sharing

posicionando assim fica mais limpo de ver( ortogonal, em colunas e sem cruzamentos)

sugestão 1:
2 diagramas
um simplificado assim e um mais completo como o atual.

sugestão 2:
A parte do preprocessing ficaria mais poderosa ainda com a ideia do pycaret (https://github.com/pycaret/pycaret/blob/master/pycaret/preprocess.py)
seria uma fabrica instanciando vários tipos de preprocessadores
aí o usuário passa como argumento os preprocessadores que ele quer usar e aí internamente aplicaria na ordem

(ele fez com um monte de if ) mas poderia ser um abstract factory
deixaria muito melhor o código

barbarasilveiraf commented

Obrigada pelo seu feedback Rodrigo!!

  • 👍1
teoria commented

Espero poder colaborar mais com a hermione no futuro.
Parabéns pelo projeto e por abrir com a comunidade.