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

Allow for multiple bot spawn positions to be saved and loaded #225

Open
Cyberschorsch opened this issue Jan 4, 2022 · 4 comments
Open

Comments

@Cyberschorsch
Copy link
Contributor

Loading multiple bots previously saved on a map is a great feature! A good addition in my mind would be to be able to save and load bots just like handling nades.

This would improve practicing executes and taking map control greatly.

@Cyberschorsch
Copy link
Contributor Author

I am working on this feature now.

My first goal is to extend the .savebot and .loadbot commands to allow passing ids which specify loading bot positions from a file in a subdirectory with the map name.

Cyberschorsch added a commit to Cyberschorsch/csgo-practice-mode that referenced this issue Feb 13, 2022
Allow passing additional id to saving and loading bots which allows for having multiple bot setups per map.

Custom setup files will be placed in a subdirectory with the map id as directory name
Cyberschorsch added a commit to Cyberschorsch/csgo-practice-mode that referenced this issue Feb 13, 2022
Cyberschorsch added a commit to Cyberschorsch/csgo-practice-mode that referenced this issue Feb 14, 2022
If subdirectories do not exist, create them
@PlamAtom
Copy link

PlamAtom commented Mar 8, 2022

I am very excited, if you need someone to do testing, just let me know.

@Cyberschorsch
Copy link
Contributor Author

You can check my latest dev build:

csgo_practice_mode.zip

Example usage:

Place bots anywhere you want on the map (e.g. CT-Bots on B-Site Dust 2)
Command: .savebots ct_b_site

Now you can load the bot positions like this:

.loadbots ct_b_site

This allows for saving and loading multiple setups.

splewis added a commit that referenced this issue Mar 13, 2022
@PlamAtom
Copy link

Hey I tested the whole thing out a little bit. I would provide feedback here. I think it is unnecessary to say things that I like because it is generally an incredible progress for Practicemode that you can save multiple bot positions and easily load them without moving files on the server and without the issue that you could make a mistake and overwrite files by accident. From bots that show callouts to prefire bots or even if you want to train specific entrypaths really awesome. Thank you a lot.

"Disadvantages":

  • 1 - (Can be ignored)
    I don't think it's bad that you can load the same bots multiple times. If you have a replay where these saved bots are killed by replay bots, they will not respawn. In three months I overlappen bots like two times on accident, so this happens so rarely that it doesn't matter. You can just type .nobots and issue solved.

  • 2 - (Quality of Life improvement)
    Missing menu so you can use more complex names without a text file to remember them. For example, it makes sense for prefire bots to be split off according to individual areas so that you can load exactly the prefire bots you want, without blowing the server with a single load of all prefire-bots.

Examples:

[12] Banana Prefire-Bots (All)
[13] Adamo Prefire-Bots (All)
[14] A-Site Prefire-Bots (All)

I like it very much. I find it incredible that thanks to Splewis, I can mimic the behavior of players in demos and then have a somewhat accurate demo made ingame. On the other hand, the bots save, that's really big. Therefore, a big warm heartfelt thank you to both of you at this point.

Maybe someday I'll publish my practice data so everyone can use it but that will definitely take a long time because I'm an extreme perfectionist. Thank you very much again.

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

2 participants