search

Sizzzles / TFT-OCR-BOT

TFT OCR Bot with auto comps loading from lolchess
GNU General Public License v3.0
18 stars 2 forks source link

Error when launching main.py #4

Closed Lexorily closed 1 year ago

Lexorily commented 1 year ago

image

Sizzzles commented 1 year ago

Yes bot is currently broken due to lolchess updating their site so it's unable to pull comps at the moment. I'll try and fix this as soon as I can but have no eta

Adding help wanted tag incase someone wants to fix it before I do

Sizzzles commented 1 year ago

Comps should be fixed now. Let me know if you are still having problems :)

diivien commented 1 year ago

Hi, no comps are showing or loaded. image

Sizzzles commented 1 year ago

Okay guess it loves breaking after I fix. I'll look into it. I'm getting the same thing

huy4vn commented 1 year ago

Hi @Sizzzles i think we should use their API. i found an API here to get the comps : https://tft.dakgg.io/api/v1/team-builders/xxx. With xxx is deck Id

Sizzzles commented 1 year ago

Do they have documentation for api? since I need more than just the team builder to make it work

huy4vn commented 1 year ago

No they not public their API, I just try to get from inspect . I read the code, you need to get the comps name too right ? I think when we first loading the https://lolchess.gg/meta we will also saving the comps name and their ids, then we will using this API to get the champ and items

huy4vn commented 1 year ago

By the way, i found another one. I think we could use this https://tft.dakgg.io/api/v1/meta-decks?hl=en

Sizzzles commented 1 year ago

Yeah didn't think it was public but thank you so much I've been looking for their API but failed to find it. Honestly if you find anymore that may be useful please let me know

huy4vn commented 1 year ago

Yeah, it bad to use load their web and parsing, API is better ^^

Sizzzles commented 1 year ago

Little update. I currently have it working with the team builders API you linked. Haven't had a chance to test it completely but comps are loading just need to run through a tft to make sure it all works correctly since I changed a bit and new requirements are needed but that will all be included when I push the changes

huy4vn commented 1 year ago

Cool ^^.

huy4vn commented 1 year ago

anything news ?

Sizzzles commented 1 year ago

Still testing. It works but had an issues with where it was placing units which I'm testing now. If it works I'll push it all before I sleep

Sizzzles commented 1 year ago

Pushed update. If you run into any problems let me know You will need to run pip install -r requirements.txt in Command Prompt again since there are new requirements If you run into trouble follow install instructions from https://github.com/jfd02/TFT-OCR-BOT

Sizzzles commented 1 year ago

Marking as resolved. But will keep open for a few days in case issues arise