DIM icon indicating copy to clipboard operation
DIM copied to clipboard

Better stats available doesn't take into account ignored stats

Open bhollis opened this issue 1 year ago • 5 comments

This is interesting - this loadout doesn't say "better stats available", but there is a set with the same values for all included stats and more points in ignored stats. Should we tweak the algorithm to find these?

Screenshot 2023-12-23 at 4 53 55 PM Screenshot 2023-12-23 at 4 53 29 PM

bhollis avatar Dec 24 '23 00:12 bhollis

It'd be neat but it's a can of worms - we'd have to extend our sorting to include ignored tiers too and LO SetTracker set sorting is already a bit shaky with the way we predict the stats string. It's also not entirely clear where in the lexicographic set comparison the "total tier" should be (vs. the total enabled tier), see https://discord.com/channels/316217202766512130/1174038965108613211/1174046151100604466 for some more discussion.

robojumper avatar Dec 24 '23 22:12 robojumper

definitely anti-including ignored stats, maybe ever, or at least not until there's either a snooze button, or some hypothetical advanced priority selection

nev-r avatar Jan 01 '24 23:01 nev-r

But why not? If it meets all the other objectives of the loadout but also has more of some stat, wouldn't it be worth it to swap out a piece to make that happen?

bhollis avatar Jan 02 '24 00:01 bhollis

because it's a bad feature if harasses me into accepting a stat tier i don't want, precluding later identification of a stat tier i wanted

nev-r avatar Jan 02 '24 02:01 nev-r

I don't think it should preclude a better result later - improvements in the enabled stats should always take priority

bhollis avatar Jan 02 '24 02:01 bhollis