HEKPOMAHT 2,180 #475670 Posted January 25, 2020 @ktulho добавь двухстволки в {{py:sight.isFullClipShells}} Quote Share this post Link to post Short link Share on other sites
ktulho 5,793 #475693 Posted January 25, 2020 @HEKPOMAHT А с перезарядкой у двухстволки все нормально? Quote Share this post Link to post Short link Share on other sites
HEKPOMAHT 2,180 #475695 Posted January 25, 2020 2 минуты назад, ktulho сказал: А с перезарядкой у двухстволки все нормально? на цифры внимания не обращал. Применил {{py:isDualGunVehicle}} - работает, {{py:sight.isFullClipShells}} - не работает. Сыграю бой - гляну как оно. Quote Share this post Link to post Short link Share on other sites
HEKPOMAHT 2,180 #475696 Posted January 25, 2020 @ktulho да, перезарядка шалит вот для теста реплей: replay_last_battle.wotreplay ---- там же по идее {{py:sight.isAutoReload}} должен срабатывать , а он не срабатывает ---- {{py:sight.leftTimeShot%3.1f}} - пусто отдаёт Quote Share this post Link to post Short link Share on other sites
ktulho 5,793 #475697 Posted January 25, 2020 (edited) 1 час назад, HEKPOMAHT сказал: там же по идее {{py:sight.isAutoReload}} должен срабатывать , а он не срабатывает Там все по другому. Надо подумать, какие макросы, что должны возвращать. 9 часов назад, HEKPOMAHT сказал: @ktulho добавь двухстволки в {{py:sight.isFullClipShells}} Поправил. Скрипт подходит для тестового клиента. quantityShells.py Edited January 25, 2020 by ktulho 1 Quote Share this post Link to post Short link Share on other sites
HEKPOMAHT 2,180 #475698 Posted January 25, 2020 57 минут назад, ktulho сказал: Поправил. Скрипт подходит для тестового клиента. бодрячком 57 минут назад, ktulho сказал: Надо подумать, какие макросы, что должны возвращать. так там тоже ж авто. Может сделать типо такого: {{py:sight.isAutoReload}} возвращает 'auto' для техники с механизмом дозарядки, 'dual' для друхстволки, иначе пусто. Quote Share this post Link to post Short link Share on other sites
ktulho 5,793 #475699 Posted January 25, 2020 6 минут назад, HEKPOMAHT сказал: Может сделать типо такого: {{py:sight.isAutoReload}} возвращает 'auto' для техники с механизмом дозарядки, 'dual' для друхстволки, иначе пусто. Это надо? auto не достаточно? Quote Share this post Link to post Short link Share on other sites
ktulho 5,793 #475700 Posted January 25, 2020 Исправлен макрос {{py:sight.quantityInClipShells}} для двухсвтольной техники. Скрипт подходит для тестового клиента. quantityShells.py Quote Share this post Link to post Short link Share on other sites
HEKPOMAHT 2,180 #475701 Posted January 25, 2020 4 минуты назад, ktulho сказал: Это надо? auto не достаточно? ну хуже от этого не будет, хотя и {{py:isDualGunVehicle}} можно применить, чтобы двухстволки (если кто захочет) выделить, например для текстовых подстановок. Но это будет усложнение конструкции. Для тех, кто особо не вникает, то и одного auto хватит Хотя как по мне, лучше разделить почини ещё для двухстволки {{py:sight.leftTimeShot}} и с {{py:sight.isAutoReload}} нужно что-то решить ---- и всё таки лучше разделить Quote Share this post Link to post Short link Share on other sites
ktulho 5,793 #475702 Posted January 25, 2020 (edited) 4 минуты назад, HEKPOMAHT сказал: Но это будет усложнение конструкции. Такая подстановка не проще: {{py:sight.isAutoReload=auto?один|{{py:sight.isAutoReload=dual?два|три}}}} Плюс придется менять название макроса. Edited January 25, 2020 by ktulho Quote Share this post Link to post Short link Share on other sites
HEKPOMAHT 2,180 #475703 Posted January 25, 2020 1 минуту назад, ktulho сказал: Такая подстановка не проще: {{py:sight.isAutoReload=auto?один|{{py:sight.isAutoReload=dual?два|три}}}} так а я тебе про что толкую: 27 минут назад, HEKPOMAHT сказал: Может сделать типо такого: {{py:sight.isAutoReload}} возвращает 'auto' для техники с механизмом дозарядки, 'dual' для друхстволки, иначе пусто. Quote Share this post Link to post Short link Share on other sites
ktulho 5,793 #475704 Posted January 25, 2020 1 минуту назад, HEKPOMAHT сказал: так а я тебе про что толкую: Про что? Quote Share this post Link to post Short link Share on other sites
HEKPOMAHT 2,180 #475705 Posted January 25, 2020 3 минуты назад, ktulho сказал: Про что? 32 минуты назад, HEKPOMAHT сказал: Может сделать типо такого: {{py:sight.isAutoReload}} возвращает 'auto' для техники с механизмом дозарядки, 'dual' для друхстволки, иначе пусто. ---- 9 минут назад, ktulho сказал: Плюс придется менять название макроса. может так: {{py:sight.isMultiReload}} Quote Share this post Link to post Short link Share on other sites
ktulho 5,793 #475706 Posted January 25, 2020 Во-первых, is в названии как бы намекает на два варианта ответа да или нет, а не на три варианта. Во-вторых, повторяю вопрос, это надо? Чем конструкция {{py:sight.isAutoReload=auto?один|{{py:sight.isAutoReload=dual?два|три}}}} лучше {{py:sight.isAutoReload?один|{{py:sight.isDualGunVehicle?два|три}}}} ? Quote Share this post Link to post Short link Share on other sites
HEKPOMAHT 2,180 #475707 Posted January 25, 2020 6 минут назад, ktulho сказал: ? на один скрипт меньше (не понадобится infoMyVehicle.py) Quote Share this post Link to post Short link Share on other sites
ktulho 5,793 #475708 Posted January 25, 2020 1 минуту назад, HEKPOMAHT сказал: на один скрипт меньше (не понадобится infoMyVehicle.py) А лучше перенести макрос {{py:sight.isAutoReload}} в infoMyVehicle.py Quote Share this post Link to post Short link Share on other sites
HEKPOMAHT 2,180 #475709 Posted January 25, 2020 1 минуту назад, ktulho сказал: А лучше перенести макрос {{py:sight.isAutoReload}} в infoMyVehicle.py тут уже сам смотри, как проще будет это ещё: 31 минуту назад, HEKPOMAHT сказал: почини ещё для двухстволки {{py:sight.leftTimeShot}} Quote Share this post Link to post Short link Share on other sites
ktulho 5,793 #475710 Posted January 25, 2020 1 минуту назад, HEKPOMAHT сказал: это ещё: Там много чего надо сделать. Quote Share this post Link to post Short link Share on other sites
HEKPOMAHT 2,180 #475711 Posted January 25, 2020 1 минуту назад, ktulho сказал: Там много чего надо сделать. ладно тогда. Отложим на потом Quote Share this post Link to post Short link Share on other sites
aster 53 #475714 Posted January 25, 2020 (edited) тогда уж ".ReloadType" Кто знает, что ещё накрутят разрабы в будущем. Edited January 25, 2020 by aster Quote Share this post Link to post Short link Share on other sites