arena21 / idps

Automatically exported from code.google.com/p/idps
0 stars 0 forks source link

Lacking native support for +34 Crit JC gem #33

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Click a socket slot
2. Notice that you cannot find the +crit Dragon's Eye
3. Feel sad

What is the expected output? What do you see instead?
I assume this is unintentional because in some situations (including the
gearset I am analyzing) Crit approaches, equals, or even surpasses the EP
of both AP and Agility. Thus, you might want to use your Dragon's Eyes on
Crit. Also, there is support for the +20 Crit non-JC gem, and the
less-likely +34 Hit gem, indicating support for all Rogue-relevant JC gems
was intended.

What version of the product are you using? On what operating system?
Version 0.14-r81 on Windows XP Home

Please provide any additional information below.
I used your FAQ instructions re: how to edit the .jar file, and was
successful in producing a working iDPS that includes the Smooth Dragon's
Eye and properly recommends it when it is appropriate based on EP
valuation. If you wish to use my amended gems.xml, I attach a copy here.

Original issue reported on code.google.com by sendinga...@gmail.com on 2 Mar 2010 at 9:30

Attachments:

GoogleCodeExporter commented 8 years ago

Original comment by ch...@clan-fwd.com on 26 Mar 2010 at 6:34