Нормальная логика. Только неоптимально. Вторые Секвенсеры лишние. Переменные "№ СВЕТОДИОД ТАЙМЕР" и "№ СВЕТОДИОД ГЕНЕРАТОР" можно взять с выходов первых секвенсеров. Блоками FreeLog надо пользоваться обосновано. Обычно можно обойтись без них. Знаете, в программировании есть понятия "the best practice" и "bad practice". Так вот в языке FBD блоки свободной логики это "bad practice" - плохой тон. Это как оператор GOTO в процедурных языках - не рекомендован к применению вообще. Представляете парадокс - оператор есть, но пользоваться им не рекомендуется
Так вот, если вы увидели у себя проекте блок свободной логики - это сигнал тревоги. Надо пересмотреть проект.
Но можно и с ним. В обоснованных случаях. Это я просто топлю за красоту и гармонию инженерного труда