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

Space characters removed from bot names 🙀 #6

Open
artkirienko opened this issue May 14, 2020 · 4 comments
Open

Space characters removed from bot names 🙀 #6

artkirienko opened this issue May 14, 2020 · 4 comments

Comments

@artkirienko
Copy link
Contributor

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

  • HLDS Build 7882.

    Protocol version 47/48
    Exe version 1.1.2.2/Stdio (valve)
    Exe build: 17:23:32 May 24 2018 (7882)
    
  • Metamod-p version 1.21p38

  • AMX Mod X version 1.8.2

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

@josemam
Copy link
Contributor

josemam commented May 20, 2020

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
Copy link
Contributor Author

@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
Copy link
Contributor

josemam commented May 20, 2020

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
Copy link

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants