search

labrax / socialtoolkit

0 stars 0 forks source link

change from networkx to graph-tool #1

Open fmcardoso opened 7 years ago

fmcardoso commented 7 years ago

Use graph-tool as background network module.

labrax commented 7 years ago

Felipe, tem um plano do que você quer testar?

Fazer essa mudança é reconstruir a ferramenta. Não é melhor fazer a conversão quando precisar do graphtool?

Podemos mudar isso, mas planejo que de umas boas horas de trabalho (30+) para mudar tudo. Talvez seja melhor fazer interfaces para evoluir a ferramenta, e, a princípio, se for mudar isto continuar usando o networkx ao mesmo tempo, atualizando no outro grafo as mudanças.

Abraço, Victor

On 25 Jan 2017 10:06 a.m., "Felipe M Cardoso" notifications@github.com wrote:

Use graph-tool as background network module.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/labrax/socialtoolkit/issues/1, or mute the thread https://github.com/notifications/unsubscribe-auth/AAiEHfOtfVEua643LYUKKBXX_XIfjgMRks5rVzqrgaJpZM4LtaE- .

fmcardoso commented 7 years ago

Eu acho que mudança para graph-tool ou igraph é fundamental para a ferramenta. Networkx afeta muito o tempo de processamento e postergar a mudança é deixar o problema mais complicado no futuro. Eu acho melhor que essa mudança seja feita antes de que seja inviável fazer a alteração.

O ideal seria conseguir desacoplar essas alterações suficientemente da interface, já que uma parte não deveria afetar a outra. Isso nos ajudaria também em uma possível situação futura onde seja necessário uma nova alteração do backend.

Eu posso me encarregar dess issue, assim voce tem mais liberdade para se concentrar na interface (que não domino nenhum pouco).