ff14-advanced-market-search / AzerothAuctionAssassin

Apache License 2.0
4 stars 6 forks source link

get_ilvl_items is called too much #69

Closed cohenaj194 closed 3 weeks ago

cohenaj194 commented 3 months ago

This makes a direct call to saddlebag for every item in an ilvl list.

https://github.com/ff14-advanced-market-search/AzerothAuctionAssassin/blob/main/utils/api_requests.py#L102C5-L119

This is excessive as theres no reason to get the same info again and again for each item.

https://github.com/ff14-advanced-market-search/AzerothAuctionAssassin/blob/main/utils/mega_data_setup.py#L362-L366

https://github.com/ff14-advanced-market-search/AzerothAuctionAssassin/blob/main/utils/mega_data_setup.py#L322

I want to fix this and reduce the load on the saddlebag api