search

SimCMinMax / SimPermut

This repository is for the addon SimPermut which mak a string usable by SimulationCraft with all the different permutations for the items you have in your inventory. This makes easy comparing stuff.
GNU General Public License v3.0
5 stars 5 forks source link

Wrong ilevel #6

Closed Llamatron2112 closed 7 years ago

Llamatron2112 commented 7 years ago

When doing a sim including an ilvl 915 Stormrage Signet Ring (http://www.wowhead.com/item=150524/stormrage-signet-ring) from the Black Temple timewalking event, the detected ilevel is wrong in SimPermut.

For that item, the official simc addon outputs: finger1=,id=150524,enchant_id=5427,bonus_id=3589/3590,drop_level=110 and the item shows as ilevel 915 as it should in the gear list

and SimPermut: finger1=,id=150524,enchant_id=5427,bonus_id=3589/3590 resulting in a ilevel 151 item in the gear list

Kutikuti commented 7 years ago

Hi, this will be added in the next update. Unfortunatly i'm currently on vacation, so i can't test what i wrote. If you want, you can download the github version of the addon and test it.

Llamatron2112 commented 7 years ago

This has fixed this issue apparently, thanks for being that fast :)

Kutikuti commented 7 years ago

just tracing it : there is an issue if the item ilvl (base, not corrected with drop_level) is not in the ilvl threshold, it will not be listed.

Llamatron2112 commented 7 years ago

I'm up for testing changes if you need.