Skip to content
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

Chatter fixes and improvements #560

Merged
merged 21 commits into from
May 26, 2024
Merged

Conversation

Vladislav4KZ
Copy link
Collaborator

Pull Request for tracking the progress of bot chatter fixes

@Vladislav4KZ
Copy link
Collaborator Author

While working on the chatter, I noticed that when deleting phrases for the Radio_NeedBackup event, the bots first use the radio command Need backup! and then spam with Hold this position!.
While doing so, though, I didn't see a single line in the code using this radio command:
pushRadioMessage (Radio::HoldThisPosition).

Пока работал над чаттером, я заметил, что при удалении фраз для события Radio_NeedBackup боты сначала используют радиокоманду Need backup!, а затем спамят командой Hold this position!.
Хотя при этом я не увидел в коде ни единой строки с использованием этой радиокоманды:
pushRadioMessage (Radio::HoldThisPosition)

@jeefo
Copy link
Collaborator

jeefo commented May 2, 2024

They're spamming in response ? or just a the bot who's issuing Need Backup ?

@Vladislav4KZ
Copy link
Collaborator Author

The bot that spams with the radio command Hold this position! is the one that used the radio command Need backup!.

Спамит радиокомандой Hold this position! именно тот бот, который использовал радиокоманду Need backup!.

@jeefo
Copy link
Collaborator

jeefo commented May 2, 2024

Поставил в общем 20 ботов после 1го, сделал их зомби, бот побежал за мной, много орал про needbackup, но ни разу hold this position. как воспроизвести?

@Vladislav4KZ
Copy link
Collaborator Author

Vladislav4KZ commented May 2, 2024

Поставил в общем 20 ботов после 1го, сделал их зомби, бот побежал за мной, много орал про needbackup, но ни разу hold this position. как воспроизвести?

Я никаких особых действий не делаю, просто вижу как только бот использует команду Need backup, так после неё либо сразу спамит командой Hold this position, либо повторяет Need backup и затем начинает спамить Hold this position.

В игре никакие моды не установлены, проверяю на стандартных картах.

jeefo added a commit that referenced this pull request May 2, 2024
chatter: increase minimum voice pitch
radio : probably fix for cycling ``hold this position`` message (ref #560)
@jeefo
Copy link
Collaborator

jeefo commented May 2, 2024

Попробуй с исправлениями в мастер ветке (в botlib.cpp), по идее, должно помочь.

@Vladislav4KZ
Copy link
Collaborator Author

Попробуй с исправлениями в мастер ветке (в botlib.cpp), по идее, должно помочь.

Всё равно после нескольких Need backup проскакивает радиокоманда Hold this position.

jeefo added a commit that referenced this pull request May 7, 2024
@Vladislav4KZ
Copy link
Collaborator Author

Всё равно после нескольких Need backup проскакивает радиокоманда Hold this position.

С коммитом 6663cda баг не воспроизводится.

@Vladislav4KZ Vladislav4KZ marked this pull request as ready for review May 26, 2024 06:39
@Vladislav4KZ Vladislav4KZ requested a review from jeefo May 26, 2024 06:39
@Vladislav4KZ
Copy link
Collaborator Author

Большую часть того, что я хотел сделать – сделал.
Чтобы не держать этот Pull Request открытым целую вечность, я думаю его уже можно мержить.

А с другими мелочами попозже разберусь.

@Vladislav4KZ
Copy link
Collaborator Author

Vladislav4KZ commented May 26, 2024

Что я сделал:

  • Удалил фразы с ивента Radio_NeedBackup, так как они не подходят по интонации, и случаям, при которых бот воспроизводит эти фразы.
  • Исправил в коде определение для чаттер ивента Chatter_HeardEnemy из chatter.cfg, который раньше считался набором фраз для ивента ScaredEmotion.
  • Добавил новый чаттер ивент Chatter_GuardingEscapeZone – с фразами, которые будет произносить бот при охране зоны побега террористов на es_ картах. Если конкретнее, то бот будет их произносить при вызове радиокоманды Report in, team
  • Добавил новый чаттер ивент Chatter_GoingToGuardEscapeZone – с фразами, которые будет произносить бот, когда он только начал охранять зону побега террористов.
  • Добавил новый чаттер ивент Chatter_GoingToGuardRescueZone – с фразами, которые будет произносить бот при охране зоны спасения заложников.
  • Исправил баг, при котором боты говорили что спасают заложников, находясь на узле с флагом RESCUE. Вместо этого, сделал условие, чтобы боты произносили эти фразы, когда они действительно ведут за собой заложников.
  • Убрал игнорирование радиокоманды Report, in team когда у бота есть заложник, бомба C4, или ставит/обезвреживает бомбу.
  • Исправил условие в коде, при котором бот должен говорить, что он слышит врага.
  • Удалил фразы с ивентов Radio_FollowMe и Radio_EnemySpotted т.к. они вообще не подходили к этим радиокомандам.
  • Добавил новые чаттер ивенты, с подсчётом количества врагов – Chatter_SpottedOneEnemy, Chatter_SpottedTwoEnemies, Chatter_SpottedThreeEnemies и Chatter_TooManyEnemies.
  • Добавил новый чаттер ивент Chatter_OnARoll – с фразами, которые будет произносить бот при совершении серии убийств из 3 или более врагов в течение 5 секунд.

@Vladislav4KZ Vladislav4KZ merged commit 54e7eb4 into master May 26, 2024
12 checks passed
@Vladislav4KZ Vladislav4KZ deleted the chatter-fixes-and-improvements branch May 26, 2024 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants