hectorgimenez / koolo

Diablo II Resurrected bot written in Go
MIT License
80 stars 51 forks source link

support for `level` in NIP files #318

Closed zazzyzeph closed 2 months ago

zazzyzeph commented 3 months ago

Details

I'm not entirely sure if this is a problem with koolo, or the NIP parser in the d2go repo but I would like to be able to have the bot keep ilvl 91+ grand charms (from diablo, baal, or TZ monsters mlvl >= 91) so i can try re-rolling them into the elusive vita skillers.

In theory, from this NIP guide, [level] can be used as a property, but when i start the Koolo exe the NIP parser returns an error on:

[name] == grandcharm && [quality] == magic && [level] >= 91

image

using it as a [stat] (ie after a #) doesn't seem to work (as that guide says, it won't). I tried it and although it didn't trigger an error when koolo starts, i caught the bot skipping over a GC dropped by Diablo.

Thanks so much for your work on this! Eases up on the SSF grind for sure and it works great :)

Version

dev 37bebc0507aaa8a7e8031455a3b87909591b9d6c

Logs

logs don't have any pertinent information for this problem

Millionaar commented 3 months ago

I haven't looked into it but I'm 90% sure it used to work just fine with Koolo. So it's likely some of the reference files in Koolo or D2GO have changed, or the D2R patch somehow broke something. In the meantime I filter to pickup charms that have the lvl91 suffix itself i.e. +36 to life. But get far fewer.