Bots-United / jk_botti

Bot (Computer AI) for Half-Life 1 Deathmatch
http://forums.bots-united.com/forumdisplay.php?f=83
Other
43 stars 19 forks source link

Space characters removed from bot names ๐Ÿ™€ #6

Open artkirienko opened 4 years ago

artkirienko commented 4 years ago

Hi @jkivilin @Bots-United! Thanks for your great work! ๐ŸŽ‰

jk_botti_names.txt

jk botti names

Screenshots

Screen Shot 2020-05-14 at 16 39 48 Screen Shot 2020-05-14 at 16 40 23 Screen Shot 2020-05-14 at 17 13 15

Versions and configs

jk_botti version 1.43

Available as a Docker Image here: https://github.com/artkirienko/hlds-docker-dproto

josemam commented 4 years ago

You might be interested to know that this behaviour is inherited from the HPB-Bot template 4. Nevertheless, it has been worked around in at least one bot based on it: FoxBot bots do keep spaces in their names. So this is probably fixable.

artkirienko commented 4 years ago

@josemam long story short, bot name format is being checked and fixed in two places: when reading names from file and when creating bot. Iโ€™ve just commented out both of these checks and rebuilt jk_botti for my server, so it supports now bot names with spaces included.

Iโ€™m not sure if it is the best solution.

Do jk_botti need to check input for bot names or game server admin needs to be responsible for correct bot names?

josemam commented 4 years ago

I have no clue on that, but I guess that the code performs other, necessary checks (e.g. for characters like รฉ รฑ รง) and only the space trimming can (probably) be safely removed.

You (we) should get advice from someone that has actually worked on bots and/or the HL engine in order to be sure on what to do here. Maybe the members of Bots-United can help.

Vladsmesh commented 9 months ago

How many lines in jk_botti_names does the plugin support? I have written more than 300 names for bots, but it seems that no more than 20 are used.