marcinn
commented
3 years ago Po głębszych testach jednak nie podoba mi się jak to działa. Przy przejściu do dwukabinowego loka pulpit staje się dysfunkcyjny (nieintuicyjnie zmienia światła w kabinie przeciwnej). Prowadzi mnie to do zastosowania logiki użytej przez Was, co w efekcie zakończy się zduplikowaniem części istniejących poleceń dotyczących przednich świateł.
Sprawdzę pierwotny pomysł bazujący tylko na dodaniu brakujących poleceń enable/disable dla rear i redmarks. Nie zamykam PR, ale odezwę się dopiero po gruntownym przetestowaniu.
WItam serdecznie. To mój debiut. Zauważyłem że nie ma możliwości sterowania światłami w lokach jednokabinowych (typu sm42) za pomocą poleceń, co utrudnia wykonanie pulpitu odzwierciedlającego SM42. Ten PR jest próbą zaimplementowania tej mechaniki.
Zdecydowałem się na zestaw dedykowanych poleceń, ponieważ nie chciałem naruszać logiki istniejących switchy.
Wykonłem pozytywne ścieżki testów. Jestem przed sprawdzaniem działania w pozostałych lokomotywach. Z tego powodu można uznać, że PR poszedł za wcześnie, ale ze względu na moje pierwsze kroki uznałem, że im wcześniej rzuci ktoś z Was okiem, tym będzie lepiej. Może w ogóle nie chcecie tego w swoim repo - też dobrze byłoby wiedzieć :)
Wątek na forum poruszający ten temat zaczyna się tu: https://eu07.pl/forum/index.php/topic,28460.msg541783.html#msg541783