search

mvk-team42 / Veracitor

An application that studies the trust in a network of users, sources and articles
6 stars 2 forks source link

Tidaltrust #4

Closed antonaut closed 11 years ago

antonaut commented 11 years ago

@mold

Finns det ngn speciell anledning till att Tidaltrust ligger i en klass och är statiska metoder istället för bara vanliga funktioner :question:

Man importerar ju iaf hela modulen, så istället för 

from tidaltrust import Tidaltrust
...
Tidaltrust.compute_trust(...)

blir det:

import tidaltrust

tidaltrust.compute_trust(...)

:smile:

JonathanMurray commented 11 years ago

Bra poäng! :+1: Enda vitsen med en klassbaserad approach är väl att kunna ha flera TidalTrust samtidigt antar jag. (men det kanske går på nåt vis ändå, mha trådar o grejer?) Nu har ju den potentiella fördelen ändå negerats av att alla metoder är static :P

mold commented 11 years ago

Bra fråga. Vi gjorde nog inget medvetet val. Vi får utvärdera strukturen senare, speciellt med tanke på parallella processor osv osv.

Just nu antar jag att vi kan fortsätta på samma sätt då det inte är problematiskt?

mrunelov commented 11 years ago

Mjo vi lär ju inte ha något arv dock, så klassen behövs inte annat än för att signalera att metoderna hör ihop (funktioner i en modul kan ju faktiskt ha lite utspridda syften :p). Men jag tror på Modul i slutändan!

mrunelov commented 11 years ago

Vi har bytt till modul med funktioner nu. Väntar med att resolva tills vi har rett ut det där med parallella körningar osv.