4rtzel / poe-archnemesis-scanner

Tool for Path of Exile game to automatically scan Archemesis inventory and display related information
Apache License 2.0
64 stars 25 forks source link

suggestion about picture #36

Open bakaheiye opened 2 years ago

bakaheiye commented 2 years ago

just keep the up half of pictures(for compare) only. In my test, it will help a lot.

meepen commented 2 years ago

I have done this in #35

kevindevm commented 2 years ago

@meepen i just tested your in 1080p and and its cool what you did about keeping 1 per slot, although i got worst result that i did in this release lots of unnacounted

meepen commented 2 years ago

What settings are you using? And can you screenshot your game for testing?

On Mon, Feb 14, 2022, 8:39 PM kevindevm @.***> wrote:

@meepen https://github.com/meepen i just tested your in 1080p and and its cool what you did about keeping 1 per slot, although i got worst result that i did in this release lots of unnacounted

— Reply to this email directly, view it on GitHub https://github.com/4rtzel/poe-archnemesis-scanner/issues/36#issuecomment-1039759417, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABPGKSNQE7K47WQXUN7H7WTU3GVGJANCNFSM5OKEWEKQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

kevindevm commented 2 years ago

@meepen https://youtu.be/S3llFLRTXiw https://i.imgur.com/8fKJ7Ey.png

meepen commented 2 years ago

I've checked with the most recent commit from that branch and the image you provided and it seems to work on my machine, at least. Can you try updating to the most recent commit?

kevindevm commented 2 years ago

I've checked with the most recent commit from that branch and the image you provided and it seems to work on my machine,

i mean i just downloaded ur fork isn't that included in there? i will try again

kevindevm commented 2 years ago

@meepen ok now it work like a charm. everything accounted for still would be cool if it show the total so i dont have to count all

and can you fix the name Empowered Minions → Empowering Minions and fix the copy to clipboard to replace the " " to "\s" becouse wont work if a space is in ( )

anyhow thanks a lot