aeon0 / d4lf

Diablo 4 Loot Filter
MIT License
154 stars 38 forks source link

Aspect recorgnized as Affix #354

Closed tikalein closed 1 month ago

tikalein commented 1 month ago

Version 5.7.1

2024-08-13 | 11:30:16.957 | ThreadPoolExecutor-0_16 | WARNING | src.item.descr.find_affixes:125 | Affix does not exist: cleaned_str='movement speed skeletal priests empower your skeletal warriors’ attacks to deal shadow damage and have a chance to stun enemies for seconds.' ||| combined_lines='+11.5% movement speed [8.0 - 12.5]% imprinted: skeletal priests empower your skeletal warriors’ attacks to deal shadow damage and have a 14.0% [5.0 - 20.0]% chance to stun enemies for 1.5 seconds. (necromancer only) requires level 59 account bound sell value: 26,387 ® durahiline 20/100,'

Aspect / Affix Seperation does not work correctly, checked on previous version pretty sure "Refactored tempered detection" causing the issue

chrisoro commented 1 month ago

Please provide a screenshot of your item, including your settings (resolution, font, upscaling, hdr). Can also hop on our discord to post a bug reports there

tikalein commented 1 month ago

image 2024-08-13 | 19:13:36.365 | ThreadPoolExecutor-0_21 | WARNING | src.item.descr.find_affixes:125 | Affix does not exist: cleaned_str='movement speed skeletal priests empower your skeletal warriors’ attacks to deal shadow damage and have a chance to stun enemies for seconds.' ||| combined_lines='+9.5% movement speed [8.0 - 12.5]% imprinted: skeletal priests empower your skeletal warriors’ attacks to deal shadow damage and have a 17.0% [5.0 - 20.0]% chance to stun enemies for 1.5 seconds. (necromancer only) requires level 80 account bound sell value: 32,581 # durability: 100/100'

1920x1080,HDR off, Font small changing to medium does not matter, no upscaling its not one single item its many

image 2024-08-13 | 19:10:18.794 | ThreadPoolExecutor-0_6 | WARNING | src.utils.ocr.read:51 | img provided to image_to_text() is empty!

Version 5.6.4 no issues with all those items

chrisoro commented 1 month ago

Thank you. Have fixed the issues and added your two images to unit tests so it doesn't happen again.