@fsharp_flood - страница 2556

Vasily

https://github.com/fsprojects/Fabulous/pull/771

Vasily

Простой как три копейки

Vasily

Но требует допиливания напильником

Vladislav

Can it run Crysis?

Dmitry

!!!!!!!

Vasily

Чо не так?

Vasily

Поправим, но потом

Ayrat

Поправим, но потом

за тебя это IDE правит

Vladimir

да, я тоже бы сразу отклонил из-за пробелов

Ayrat

ну там райдер например

Ayrat

да, с пробелами ад

Danil

да, с пробелами ад

Реакция на "не мой" стайлкоп

Ayrat

Реакция на "не мой" стайлкоп

ну есть обычный стайлкоп и Васин

Ayrat

то есть Вася единственный в мире фшарпист, которого забанили на пробеле

Vasily

Ну ща чаще ставлю, чем не ставлю

Ayrat

@vshapenko timer.Elapsed.Subscribe|>(fun _-> do Debug.WriteLine "Updating on timer" program.syncAction (fun()->updateView lastModel) ()) |> ignore что происходит в updateView если lastModel не менялся?

Ayrat

просто раньше updateView вызывался по требованию, а сейчас 60 раз в секунду на похуй

Vladimir

там же AutoReset false стоит

Vasily

@vshapenko timer.Elapsed.Subscribe|>(fun _-> do Debug.WriteLine "Updating on timer" program.syncAction (fun()->updateView lastModel) ()) |> ignore что происходит в updateView если lastModel не менялся?

Ничего не происходит

Ayrat

там же AutoReset false стоит

хм. и правда

Vasily

Там именно идея таймера с однократным срабатыванием

Vasily

Эдакий backpressure

Vasily

Типа апдейты проводи, но обновляй не чаще, чем раз в n миллисекунд

Ayrat

Типа апдейты проводи, но обновляй не чаще, чем раз в n миллисекунд

надо коментов намазать

Ayrat

типа с пояснением что чего делает и нахуя

Vasily

Ну там много надо намазать будет

Ayrat

потому что я сразу идею не вкурил

Vladimir

я все еще не вкурил) таймер срабатывает только один раз и больше никогда не срабатывает

Vasily

Не так

Vladimir

аа, на каждом апдейте мейлбокс создается

Vladimir

а нет)

Vladimir

хз тогда)

Vasily

https://github.com/fsprojects/Fabulous/pull/771/files#diff-b5fbbc37a7091cd902db37e045f1c70eR105-R106

Vasily

Идея такая

Vasily

Прошел апдейт с перерисовкой - запустился таймер

Vasily

Хотя даже не так

Ayrat

Можно писать многострочные коменты если что. Потому что тут неочевидно что происходит и почему это должно работать

Vasily

Да вот надо будет, да

Vladimir

а точно timer.Enabled<-true перезапустит таймер?

Vasily

Почему нет?

Vasily

Это аналогично timer.Start

Vladimir

ну я не вижу в доке чтобы это было аналогично)

Vladimir

там написано the Start method must be called in order to start the count again.

Vladimir

даже не should а must

Vasily

true if the Timer should raise the Elapsed event; otherwise, false. The default is false.

Vasily

Он внутри все равно тикает скорее всего

Vasily

просто эвент не кидает

Vasily

Я проверял, работает

Vasily

Ставишь секунду задержки, долбишь по кнопке

Vasily

Перерисовывает раз в секунду

Vasily

Если можно написать проще, велкам

Vladimir

да, посмотрел исходники - установка в false убивает таймер полностью)

Roman

а если просто старт вызвать, будет другой эффект?

Vladimir

да, вроде старт намного легковесней, он ничего не убивает) надо только проверить продолжит ли работать так как Вася хотел

Ayrat

Напишу сюда фидбек человека без контекста 1) Неясно что PR решает (было ли проблемой 60fps? где ишью, если нет ишью, то надо описать боль) 2) Неясно как решает (нет коментов, идея неясна, как работает код и почему он даёт 60fps) 3) Возможно надо гифки с ДО и ПОСЛЕ, хз 4) форматирование кода не по конвенциям

Ayrat

идея скорее всего благая, но такой PR тяжело вмёржить. Я бы попросил доработать

Vladimir

а если просто старт вызвать, будет другой эффект?

сорри, не про то ответил) старт или Enabled=true одно и то же делают

Vladimir

это Enabled=false убивает

Vasily

Напишу сюда фидбек человека без контекста 1) Неясно что PR решает (было ли проблемой 60fps? где ишью, если нет ишью, то надо описать боль) 2) Неясно как решает (нет коментов, идея неясна, как работает код и почему он даёт 60fps) 3) Возможно надо гифки с ДО и ПОСЛЕ, хз 4) форматирование кода не по конвенциям

Проблема не 60 fps, а большое число апдейтов, каждый из которых вызывал перерисовку

Vasily

идея скорее всего благая, но такой PR тяжело вмёржить. Я бы попросил доработать