New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not reduce unit morale if attack was not dangerous #220

Closed
ozkriff opened this Issue Aug 18, 2016 · 3 comments

Comments

1 participant
@ozkriff
Owner

ozkriff commented Aug 18, 2016

#191 - странно вычитать полную мораль, если юнит даже не будет останавливаться от такой атаки.

Надо по той же границе или вообще не понижать мораль, или понижать ее значительно меньше.

@ozkriff

This comment has been minimized.

Show comment
Hide comment
@ozkriff

ozkriff Sep 8, 2016

Owner

Есть мысль, что ядро во время выполнения команды должно зашивать в CoreEvent::Attack количество отнимаемого боевого духа. Оно же результаты попадания считает и знает шанс успешности атаки.

Owner

ozkriff commented Sep 8, 2016

Есть мысль, что ядро во время выполнения команды должно зашивать в CoreEvent::Attack количество отнимаемого боевого духа. Оно же результаты попадания считает и знает шанс успешности атаки.

@ozkriff

This comment has been minimized.

Show comment
Hide comment
@ozkriff

ozkriff Oct 23, 2016

Owner

Есть мысль, что ядро во время выполнения команды должно зашивать в CoreEvent::Attack количество отнимаемого боевого духа.

Эта информация там уже есть: поле suppression. Надо только в том же месте, где происходит рассчет значения remove_move_points, еще и посчитать зависимость морали от шанса нанесения урона.

Owner

ozkriff commented Oct 23, 2016

Есть мысль, что ядро во время выполнения команды должно зашивать в CoreEvent::Attack количество отнимаемого боевого духа.

Эта информация там уже есть: поле suppression. Надо только в том же месте, где происходит рассчет значения remove_move_points, еще и посчитать зависимость морали от шанса нанесения урона.

ozkriff added a commit that referenced this issue Oct 23, 2016

@ozkriff

This comment has been minimized.

Show comment
Hide comment
@ozkriff

ozkriff Oct 23, 2016

Owner

Таки это надо бло делать в другом месте - сразу где сам AttackInfo формируется.

Owner

ozkriff commented Oct 23, 2016

Таки это надо бло делать в другом месте - сразу где сам AttackInfo формируется.

@ozkriff ozkriff closed this Oct 23, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment