GICodeWarrior / fir

Foxhole Inventory Report
MIT License
20 stars 9 forks source link

additional retrain of tensorflow for new Icon mods #12

Closed softttware closed 1 week ago

softttware commented 5 months ago

I've been testing this tool out with different icon mods, while mostly work I've found an issue with this relatively newer icon mod. this is of no urgency and wanted to know if there's anything to be done because tensorflow is out of my expertise

https://rainbowbu.itch.io/new-icons-modfoxhole

tysm. <3

GICodeWarrior commented 5 months ago

Thanks for sharing. I'll add this icon mod to the training set.

I aim for 100% accuracy, so if you notice any other issues, please send me some example screenshots. I will add them to my test suite and attempt to resolve in the next update. (feel free to DM screenshots via Discord if you prefer not to post)

GICodeWarrior commented 5 months ago

@softttware please give it a try now and let me know if you notice any issues.

softttware commented 4 months ago

My apologies, I only got to this right now. It still has errors still but I think that is just a problem with resolution and contrast

GICodeWarrior commented 4 months ago

Can you DM me an example screenshot?

The tool is designed to work at all common resolutions, etc.

So if it's not working, I'd like to fix it.

On Wed, Jun 26, 2024, 4:49 PM softttware @.***> wrote:

My apologies, I only got to this right now. It still has errors still but I think that is just a problem with resolution and contrast

— Reply to this email directly, view it on GitHub https://github.com/GICodeWarrior/fir/issues/12#issuecomment-2192800526, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAE2W6C4T6DYEL4QDJWDVTZJNHPDAVCNFSM6AAAAABHNNTDMWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJSHAYDANJSGY . You are receiving this because you modified the open/close state.Message ID: @.***>

softttware commented 4 months ago

Here's a screenshot with the item mod. the items that have been causing issues are both Cometa Revolver and .44 Ammo reporting wrongly

Screenshot War-2024-06-25 22_33_08

I don't know how to DM you but if you want to reach out to me on discord i'm @softtware there.

mod link just in case: https://rainbowbu.itch.io/new-icons-modfoxhole

GICodeWarrior commented 4 months ago

Thanks for sharing.

That image seems to work correctly for me. This is what https://fir.gicode.net/ says: 2024-06-29T03_26_54 186Z_foxhole-inventory-totals

Please let me know if you see anything wrong there.

If you are running the recognition inside a spreadsheet, that spreadsheet likely needs to be updated. I put some high level instructions for updating in issue #13.

In the meantime, you could use the "Append to Google Spreadsheet" button to add data to your spreadsheet.

softttware commented 4 months ago

I will rebuild FIR and update it and will let you know!

softttware commented 3 months ago

reopening this just to ask for the discord in which I can reach out to you, sorry for the trouble

GICodeWarrior commented 1 week ago

I believe this issue was resolved. Feel free to open a new issue if needed.