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

Is there support for AHBot and Playerbots? #26

Open
Stianos-hub opened this issue Oct 4, 2024 · 16 comments · May be fixed by #20
Open

Is there support for AHBot and Playerbots? #26

Stianos-hub opened this issue Oct 4, 2024 · 16 comments · May be fixed by #20
Labels
documentation Improvements or additions to documentation help wanted Extra attention is needed question Further information is requested

Comments

@Stianos-hub
Copy link

Hey,

I've gotten everything set up and the game running, but I've been struggling to get AH BOT and Playerbot turned on. Is there an easy way to do this?

Thanks!

@Byloth
Copy link
Owner

Byloth commented Oct 4, 2024

Hi, @Stianos-hub!
Thanks for your interest into this project! 🙃

Actually, yes...
There's a PR opened about it that can't be closed yet.

All the details here: #20


I'm not sure about it but, to be precise...
If you're not using the tbc version of the server, maybe you can run it up by your own just switching branch and building the Docker images by yourself.

@Byloth Byloth linked a pull request Oct 4, 2024 that will close this issue
@Byloth
Copy link
Owner

Byloth commented Oct 4, 2024

Any help for this PR is greatly appreciated and encouraged! 🥹

@Stianos-hub
Copy link
Author

Hey, I"ll absolutely take a look. I'm attempting to get this working on classic, but I looking at the code it doesn't look to expansion specific. I see some places I missed some flips, going to try some changes and will post results!

@Stianos-hub
Copy link
Author

Have you gotten them working on the TBC servers? I've added the changes from the PRs and set it to pull the development branch, but I'm still missing something to get the commands working in game.

@Byloth
Copy link
Owner

Byloth commented Oct 5, 2024

Have you gotten them working on the TBC servers?

No, I think it's still incompatible with the tbc version of the server.

TBH, I haven't tested it in the past few days; maybe something has changed.
I will try further, later this week.

@Byloth
Copy link
Owner

Byloth commented Oct 9, 2024

I just tried to compile the source code including the latest version of CMaNGOS Playerbots and it doesn't even complete properly.
It fails with a lot of errors; it probably wouldn't make sense to paste them here...

Answering your question, then...
No. tbc version of the server is still incompatible.

@Byloth Byloth changed the title Add AH Bot and Player BOT ❔: Is there support for AHBot and Playerbots? Oct 9, 2024
@Byloth Byloth added the question Further information is requested label Oct 9, 2024
@Byloth Byloth changed the title ❔: Is there support for AHBot and Playerbots? Is there support for AHBot and Playerbots? Oct 9, 2024
@Stianos-hub
Copy link
Author

I was working on it this weekend, had to take a break for my real job, but I found that the command changed names for cmake, I was going to try and reincorporate what I learned next chance I get. Biggest things I found were:
The command is actually DBUILD_PLAYERBOTS (With an S!)
image

Running with a relative file path causes issues, and it's better to run it from the working directory.
image

That plus making sure the FullInstallDb.sh runs the commands to add the playerbot tables was what got it working for me on classic.

Once I find time I wanted to back incorporate this into the docker files and see if I can get it to work.

Hopefully some of this helps if you want to try and incorporate it into tbc before I get a chance to make some changes

@Byloth
Copy link
Owner

Byloth commented Oct 10, 2024

Yeah... No worries! 😉

I would say that's already like this:

  1. -D BUILD_PLAYERBOTS=ON \
  2. gosu mangos ./mangosd

@Byloth
Copy link
Owner

Byloth commented Oct 10, 2024

@Stianos-hub
Copy link
Author

Well you were certainly ahead of me then! I'll try and catch up so hopefully I can contribute something useful!

@Byloth
Copy link
Owner

Byloth commented Nov 14, 2024

@draew6: I just tested playerbots on tbc (using the feature/bots branch) and it worked fine.

@draew6
Copy link

draew6 commented Nov 17, 2024

 @Byloth I changed branch and did docker compose up
 
  ✘ mangosd Error pull access denied for byloth/cmangos-tbc, repository does not exist or may require 'docker login' 1.2s
✘ realmd Error context canceled

@Byloth
Copy link
Owner

Byloth commented Nov 17, 2024

Yeah, sorry... My bad!
Since this branch isn't production ready, no images are built automatically.

You have to build it by yourself.

Use...

./cmangos-build.sh --help

... for more details! 😉

@draew6
Copy link

draew6 commented Nov 18, 2024

@Byloth I had to build 2 images and it took long time but it was worth it! it's working and bots are living their dream :D didnt test anything specific yet, just interacted with them for 30 minutes but I think any other problems with them would be out of scope of this project so I think it works as expected from my side! thanks for your contribution and instructions (y)

@Byloth
Copy link
Owner

Byloth commented Nov 18, 2024

Awesome, @draew6! 🤩


[...] it took long time but it was worth it!

Yeah... CMaNGOS takes a lot of time to build! 🥲
Using multiple threads, accordingly to your hardware, might help!


[...] but I think any other problems with them would be out of scope of this project [...]

Not necessarily, actually... 🤔
May I ask you if you can report here or here any future issues or additional steps / configurations you might need in the future to improve your experience?

This could also help improve the future documentation about bots, configuration process and so on...

Thanks!

@Byloth
Copy link
Owner

Byloth commented Nov 18, 2024

I'm going to lock this conversation.
All the information you need is already here, IMHO.

From now on, please refer to the PR for the status of "Bot support" and Discussion if you have any questions or concerns.
This issue will be closed as soon as the related PR is merged.


Thank you all for your interest in this project! ❤️

Repository owner locked and limited conversation to collaborators Nov 18, 2024
@Byloth Byloth added documentation Improvements or additions to documentation help wanted Extra attention is needed labels Nov 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants