[Patch] Auction House Bot (Xeross' Branch)

cyberium · August 17, 2010

Clear mrelfire you are certainly

So you are right that's the price to be more precise.

But if i add an multiplicator variable will you be happy? (applyable for all value choosen)

EARTHWALKER · August 17, 2010

I like your idea Cyberium. Numbers rather than percentages is ALWAYS (no matter what anyone says) going to be easier in the long run, so if you can implement such a system, that would be fantastic.

Anyone who says managing percentages is easier, is either a mathematician, someone showing off or just clinically insane

DaViper · August 17, 2010

I'll back up Eathwalker there...although he could have added Massochist to that list....lol

mrelfire · August 17, 2010

I am not a mathematician

the % is just to quickly increase the "basic" rules to a biggest numbers. the multiplicator is quite a very good idea.

SkullDrago · August 18, 2010

@cyberium i have one error by pulling your ahbot

git reported me that my local changes of dbcediter/searchfrm.dfm would be overwritten and says than aborting

so i can´t make a patch file

EARTHWALKER · August 18, 2010

@cyberium i have one error by pulling your ahbot
git reported me that my local changes of dbcediter/searchfrm.dfm would be overwritten and says than aborting

so i can´t make a patch file

If you pay a little more attention and read, you'll see that a few posts up Cyberium released a patch and a download link, so you don't have to make one.

http://www.mediafire.com/file/9w43nn3lbfw69i6/AHBot_10363.patch

Unkle Nuke · August 18, 2010

git reported me that my local changes of dbcediter/searchfrm.dfm would be overwritten and says than aborting

I used to have this same issue with many repositories, until I learned how to properly configure my Git for working with cross-platform code, like the MaNGOS project.

Either you or cyberium does not have their Git properly configured to handle line endings between Unix and Windows. Depending on which OS you use, Git can interpret these line endings as changes to the code.

To prevent this issue and have Git automatically convert between the two different line ending formats use these Git commands:

* For Windows users: gitconfig --global core.autocrlf true
* For Linux/Mac/FreeBSD: git config --global core.autocrlf input

Unfortunately, this will not work on existing working trees. You will need to remove affected trees and then check them out again. You can do this by deleting all files and folders in your repository, except for the .git directory/folder, and then using git checkout <branchname> to correct line endings.

ki2007 · August 18, 2010

Hi,

I get the folowing:

http://ki2007.pastebin.com/YCFuvaXm

cyberium · August 18, 2010

-> SkullDrago : yes it's probably my fault i will try to correct my repo.

-> Unkeluke : Thanks for your good explanations

-> Ki2007, I can't reproduce your error. Be sure there is nothing wrong with your patching method or with pull/merge.

krampf · August 18, 2010

I too get the same error (using patch provided by Earthwalker):

In file included from ../../../src/game/AuctionHouseBot.cpp:1:
../../../src/game/AuctionHouseBot.h:945: error: ‘ObjectGuid’ does not name a type
../../../src/game/AuctionHouseBot.h:958: error: ‘ObjectGuid’ does not name a type
../../../src/game/AuctionHouseBot.cpp: In member function ‘void AuctionHouseBot::addNewAuctions(AHBConfig*)’:
../../../src/game/AuctionHouseBot.cpp:413: error: ‘GetAHBplayerGUID’ was not declared in this scope
../../../src/game/AuctionHouseBot.cpp: In member function ‘void AuctionHouseBot::addNewAuctionBuyerBotBid(AHBConfig*, WorldSession*)’:
../../../src/game/AuctionHouseBot.cpp:578: warning: format ‘%f’ expects type ‘double’, but argument 3 has type ‘long double’
../../../src/game/AuctionHouseBot.cpp:632: warning: format ‘%f’ expects type ‘double’, but argument 3 has type ‘long double’
../../../src/game/AuctionHouseBot.cpp:639: warning: format ‘%f’ expects type ‘double’, but argument 3 has type ‘long double’
../../../src/game/AuctionHouseBot.cpp:658: warning: format ‘%f’ expects type ‘double’, but argument 3 has type ‘long double’
../../../src/game/AuctionHouseBot.cpp:679: error: ‘GetAHBplayerGUID’ was not declared in this scope
../../../src/game/AuctionHouseBot.cpp:691: error: ‘GetAHBplayerGUID’ was not declared in this scope
../../../src/game/AuctionHouseBot.cpp:700: error: ‘GetAHBplayerGUID’ was not declared in this scope
../../../src/game/AuctionHouseBot.cpp:712: error: ‘GetAHBplayerGUID’ was not declared in this scope
../../../src/game/AuctionHouseBot.cpp: In member function ‘void AuctionHouseBot::Initialize()’:
../../../src/game/AuctionHouseBot.cpp:780: error: ‘m_FakeGuid’ was not declared in this scope
../../../src/game/AuctionHouseBot.cpp:783: warning: format not a string literal and no format arguments
../../../src/game/AuctionHouseBot.cpp:810: warning: format not a string literal and no format arguments
../../../src/game/AuctionHouseBot.cpp: In member function ‘void AuctionHouseBot::Commands(uint32, uint32, uint32, char*)’:
../../../src/game/AuctionHouseBot.cpp:1074: error: ‘GetAHBplayerGUID’ was not declared in this scope
make[3]: *** [AuctionHouseBot.o] Error 1

git apply src/bindings/ScriptDev2/patches/MaNGOS-9519-ScriptDev2.patch
git apply ../AHBot_10363.patch
autoreconf --install --force
mkdir objdir
cd objdir
../configure --prefix=/opt/mangos --sysconfdir=/opt/mangos/etc --enable-cli --datadir=/opt/mangos/data --with-python --enable-ra
make

I will pull from you Cyberium. Will let you know if it compiles.

cyberium · August 19, 2010

Oh! I didn't notice you are trying to compile it from linux. I have not set any linux environnement to test my patch for now.

I will try to find why ObjectGuid isn't declared ine your case if i have time tomorrow.

FeRkEl · August 19, 2010

maybe its just an including problem?

try to first include objectmgr.h and than auctionhousebot.h or include objectmgr.h in the auctionhousebot.h, too

(not testet, but it seems that ObjectGuids not declared before using?)

firedervil11 · August 19, 2010

yep confirmed.

Include ObjectMgr.h in auctionhouse.h will solved the Problem.

(tested under Debian Lenny)

cyberium · August 19, 2010

Hello, i pushed new tree with new version of ahbot on my repo.

I need some beta tester to improve it.

What's new :

- Completly rewrited addAuction method.

- Completly rewrited AHBConfig class.

- More selective on what item needed to be added.

- Add items by amount instead by percent.

- All items selled will be replaced by another one of same quality in next scan.

- And some more improvement like new auction speedup.

What i need?

Some feedback about object in AH. Iam not sure if it was a good idea to group itemsTradeGood with normal items. So i need to know if all usual items are still present and in good percentage.

Olso the price of items need to be checked. ( anyway you still can modify de percentage of it in DB )

Warning this version need a new database structure so i rename it to let you keep old one for reversing...

The code is probably not cleaned (trailing space, or indent problem) don't be rude it's still in devellopement

See you.

http://github.com/cyberium/mangos/commits/new_ahbot

xeross155 · August 19, 2010

Once I get my branch working again I'll merge it with cyber's branch and after that we can cooperate on it.

EARTHWALKER · August 19, 2010

'tis a great sub-development idea. I will follow this with intrigue

firedervil11 · August 20, 2010

i have tested the compile of the Branch on my Debian Lenny

and get this error.

I fix it, to test the rest again. so my question anyone else the same problem?

In file included from ../../../src/game/AuctionHouseBot.cpp:1:
../../../src/game/AuctionHouseBot.h:320: error: expected â,â or â...â before â*â token

../../../src/game/AuctionHouseBot.h:320: error: ISO C++ forbids declaration of âItemâ with no type

../../../src/game/AuctionHouseBot.cpp:70: error: prototype for âvoid AuctionHouseBot::SetPricesOfItem(const Item*, AHBConfig&, uint32&, uint32&, uint32&, e_ahb_quality)â does not match any in class âAuctionHouseBotâ

../../../src/game/AuctionHouseBot.h:320: error: candidate is: void AuctionHouseBot::SetPricesOfItem(int)

../../../src/game/AuctionHouseBot.cpp: In member function âvoid AuctionHouseBot::addNewAuctions(AHBConfig&)â:

../../../src/game/AuctionHouseBot.cpp:145: error: no matching function for call to âAuctionHouseBot::SetPricesOfItem(Item*&, AHBConfig&, uint32&, uint32&, uint32&, e_ahb_quality)â

../../../src/game/AuctionHouseBot.h:320: note: candidates are: void AuctionHouseBot::SetPricesOfItem(int)

cyberium · August 20, 2010

i have tested the compile of the Branch on my Debian Lenny
and get this error.

I fix it, to test the rest again. so my question anyone else the same problem?

Try to add #include "AuctionHouseMgr.h" in AuctionHouseBot.h

I will push it soon

xeross155 · August 20, 2010

Xeross' Branch Update

The AHBot is functional again, I'm now optimizing it, 1 single optimization got rid of 40 lines of code, after this is done and final test has been ran I will upload it.

Update 2

Everything is functioning, some bugs fixed, merging with latest mangos, after that need to check some formulas.

Update 3

Some more optimizing and renaming. merge is completed. found bugs in the formulas, fixing now.

Update 4

Testing...

mrelfire · August 20, 2010

are we going to under review for Vladimir ? with which redepository ?

xeross155 · August 20, 2010

are we going to under review for Vladimir ? with which redepository ?

Once the branches are merged than we should get under review, my branch has a lot of new optimizations and stuff, in other words better code quality than it was before.

Updated to 10385

cyberium · August 20, 2010

Just tried to merge, and faced too much conflit so i will later try to patch your repo instead to merge. My repo will be adjusted. Anyway i have not lot of time until next week so be patient

madmax765 · August 21, 2010

Once the branches are merged than we should get under review, my branch has a lot of new optimizations and stuff, in other words better code quality than it was before.
Updated to 10385

ok are we using

git clone git://github.com/xeross/mangos.git ahbot

if so i tryed to make patch file

git clone git://github.com/mangos/mangos.git
git clone git://github.com/xeross/mangos.git ahbot

cd ahbot

git checkout -b ahbot origin/ahbot

git diff master ahbot > ../ahbot.patch

Got patch file size 11,135 kb. try to apply it to core Rev. 10391 and get ton of errors.. relly to many to post here.. where did i go wrong at ?

mrelfire · August 21, 2010

1/

Once the branches are merged than we should get under review, my branch has a lot of new optimizations and stuff, in other words better code quality than it was before.
Updated to 10385

perfect did you implement the generic seller like cyberyum ?

2/ I have compiled

and also I have some strange errors on the console:

AHBOT> Running seller loop

AHBOT> Running buyer loop

AHBOT> Running seller loop

AHBOT> Running buyer loop

AHBOT> Running seller loop

AHBOT> Running buyer loop

AHBOT> Running seller loop

AHBOT> Running buyer loop

SQL: UPDATE auctionhouse SET buyguid = '135',lastbid = '946' WHERE id = '1026094

'

SQL ERROR: Table 'characters_tlk.auctionhouse' doesn't exist

auctionhouse' has been changed do we have to do something ?

Darkruler · August 21, 2010

1/
perfect did you implement the generic seller like cyberyum ?

2/ I have compiled

and also I have some strange errors on the console:

auctionhouse' has been changed do we have to do something ?

Table was renamed to "auction" in this commit

search the core for the query and change auctionhouse to auction

Sign In

[Patch] Auction House Bot (Xeross' Branch)

Recommended Posts

Link to comment

Share on other sites

Top Posters In This Topic

Popular Days

Top Posters In This Topic

Popular Days

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Recently Browsing 0 members

Contact Us

Repositories

Important Information