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.

Sign up for GitHub

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

Jump to bottom

[PlayerBot] Initial implementation of playerbots #192

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

[PlayerBot] Initial implementation of playerbots #192

wants to merge 17 commits into from

Conversation

PargeLenis
Copy link
Contributor

@PargeLenis PargeLenis commented Nov 19, 2022 •
edited
Loading

Bots still behave strangely sometimes and don't fight hard.
Randombots sometimes cause crash, will hopefully be fixed soon.
Updates will be needed for efficient fighting and bg/arena-implementation.

Bug reports are appreciated to ensure nice playability quickly.

This change is Reviewable

@billy1arm
Copy link
Member

@PargeLenis

Was this based on MangosZero or MangosOne bots ?

  • I'll need to ensure the required DB tables are also correct

@PargeLenis
Copy link
Contributor Author

PargeLenis commented Jan 18, 2023 •
edited
Loading

It was based on MangosOne and still is a little buggy.
Some tables are already loaded into the DB, see characterLoadDB.sql:

  • ai_playerbot_names
  • ai_playerbot_random_bots
    It looks like they are coming from an earlier attempt to implement the MangosZero bots.

Still missing tables:

  • ai_playerbot_custom_strategy
  • ai_playerbot_speech
  • ai_playerbot_speech_probability
  • ai_playerbot_guild_names
  • ai_playerbot_guild_tasks
  • ai_playerbot_db_store
    Loaded the structures from MangosOne src/modules/playerbot/sql for testing and totally forgot to add changes to the database repository.
    The missing tables should be loaded from characters_ai_playerbot.sql, characters_ai_playerbot_speech.sql and ai_playerbot_guild_names.sql.

Sorry for the mess, lately life has prevented me from putting more time into the development.
I'm will commit an update for the Database.

@PargeLenis PargeLenis force-pushed the playerbots branch 3 times, most recently from 77b01bd to 72e3e8f Compare January 22, 2023 20:50
@PargeLenis PargeLenis force-pushed the playerbots branch 6 times, most recently from 8107fdf to bb74c98 Compare February 7, 2023 13:18
@AppVeyorBot
Copy link

✅ Build Mangos TWO Server 22.02.47 completed (commit 8ce02b0a1f by @)

@PargeLenis PargeLenis force-pushed the playerbots branch 2 times, most recently from ddbdafc to d452ceb Compare February 10, 2023 12:39
@AppVeyorBot
Copy link

✅ Build Mangos TWO Server 22.02.48 completed (commit 237622f6a8 by @)

@AppVeyorBot
Copy link

✅ Build Mangos TWO Server 22.02.49 completed (commit 2435c533fd by @)

@AppVeyorBot
Copy link

✅ Build Mangos TWO Server 22.02.50 completed (commit 65adcf9fa5 by @)

@AppVeyorBot
Copy link

✅ Build Mangos TWO Server 22.02.51 completed (commit 93390f0e79 by @)

@AppVeyorBot
Copy link

✅ Build Mangos TWO Server 22.02.52 completed (commit 923f9ae397 by @)

@AppVeyorBot
Copy link

✅ Build Mangos TWO Server 22.02.54 completed (commit eb1447acdc by @PargeLenis)

@AppVeyorBot
Copy link

✅ Build Mangos TWO Server 22.02.55 completed (commit 0339487a61 by @PargeLenis)

billy1arm pushed a commit that referenced this pull request Oct 22, 2023
@Meltie2013 @i-am-fyre @billy1arm
Fix Trade Skill Limit (#192)
2212e1e 
* Fix trade skill max limit count
- Removed unused configuration
- Removed GM Ignore configurations

Notice: With GM Ignore configurations removed, trade skill count works as intended again. Accounts with higher permission can still learn additional trade skills for testing or debugging. As before there was no limit to how many you could learn before this commit.

Co-authored-by: Fyre <[email protected]>

* Update Configuration Version
2023031100 -> 2023053000

* Fix an incorrect check method

---------

Co-authored-by: Fyre <[email protected]>
PargeLenis and others added 9 commits October 24, 2023 15:20
@AppVeyorBot
Copy link

✅ Build Mangos TWO Server 22.02.68 completed (commit 1efde85f73 by @PargeLenis)

@PargeLenis
Copy link
Contributor Author

PargeLenis commented Oct 24, 2023 •
edited
Loading

Don't merge, still work in progress...
Just wanted to be able to use Openssl 3.x...

billy1arm added a commit that referenced this pull request Jun 23, 2024
@billy1arm
Revert "Fix Trade Skill Limit (#192)"
e69e388 
Commit broke guild creation

This reverts commit 2212e1e.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@PargeLenis @billy1arm @AppVeyorBot