Add curved smell propagation #3
Conversation
| import net.ceedubs.ficus.Ficus._ | ||
| Try(forminConfig.as[ConfigType]("config")) match { | ||
| Try(parsedConfig.as[ConfigType]("config")) match { | ||
| case Success(parsedConfig) => |
There was a problem hiding this comment.
Zmiana nazwy zmiennej wyżej jak najbardziej słuszna, ale tutaj teraz będzie shadowing zmiennej. Może w tej linijce zmienić parsedConfig na config?
|
|
||
| trait XinukConfigWithBendFactors extends XinukConfig { | ||
| def crossBendFactor: Double | ||
| def straightBendFactor: Double |
There was a problem hiding this comment.
Zastanawiam się, czy nie byłoby prościej dodać te wartości do podstawowego XinukConfigu - jeżeli symulacja nie skorzysta z zakrzywiania sygnału, to po prostu zostaną zignorowane. Idąc krok dalej, gdyby zakrzywiony zapach ustalić jako domyślny, podstawiając 0.0 pod te wartości chyba otrzymamy z powrotem oryginalny wzór bez zakrzywiania, chociaż pewnie mniej wydajnie liczony (wyliczanie zerowych składników vs. z góry pomijanie zerowych składników).
| import scala.collection.immutable.TreeSet | ||
| import scala.util.{Failure, Success, Try} | ||
|
|
||
| class SimulationWithCurvedSmellPropagation[ConfigType <: XinukConfig : ValueReader]( |
There was a problem hiding this comment.
Jeżeli dodatkowe pola konfiguracji dodamy do bazowego XinukConfigu, to unikniemy tworzenia całej tej klasy.
…strate functionallity.
2 participants
No description provided.