KaneTW / FFXIVBisSolver

FFXIV Materia-aware BiS solver
MIT License
13 stars 5 forks source link

Food not using spell seed percentage, assuming capped. #33

Closed Cadwgan closed 7 years ago

Cadwgan commented 7 years ago

To reproduce:

Calculating SMN BiS with a min speed of 850 results in the following set (http://ffxiv.ariyala.com/ZF76) claiming 861 speed, however the set actually has 848 speed.

With a base of 808 speed your tool is adding the capped +53 speed where 5% is only +40.

This may not be a massive issue with most stats but I have not tested if accuracy has the same issue.

edit: disregard, user error.

KaneTW commented 7 years ago

Are you using a recent version? I'll verify.

E: you're likely using an old version. This is the result with mine:

Gear:
        Main Hand: Mimesis Lux
        Head: Augmented Shire Philosopher's Hat
        Body: Augmented Shire Philosopher's Coat
        Hands: Alexandrian Gloves of Casting
        Legs: Augmented Shire Philosopher's Hose
        Feet: Augmented Shire Philosopher's Thighboots
        Waist: Augmented Shire Philosopher's Belt
        Ears: Augmented Shire Philosopher's Earring
        Neck: Proto Ultima Necklace of Casting
        Wrists: Alexandrian Bracelets of Casting
        Left Ring or Right Ring: Augmented Shire Philosopher's Ring
        Left Ring or Right Ring: Alexandrian Ring of Casting
Materia:
        Head: Augmented Shire Philosopher's Hat,
                 - Materia: Savage Aim Materia V
                           Amount: 2
        Body: Augmented Shire Philosopher's Coat,
                 - Materia: Quicktongue Materia V
                           Amount: 2
        Hands: Alexandrian Gloves of Casting,
                 - Materia: Quicktongue Materia V
                           Amount: 2
        Legs: Augmented Shire Philosopher's Hose,
                 - Materia: Savage Aim Materia V
                           Amount: 2
        Feet: Augmented Shire Philosopher's Thighboots,
                 - Materia: Savage Aim Materia V
                           Amount: 2
        Waist: Augmented Shire Philosopher's Belt,
                 - Materia: Quicktongue Materia V
                           Amount: 1
        Ears: Augmented Shire Philosopher's Earring,
                 - Materia: Quicktongue Materia V
                           Amount: 1
        Neck: Proto Ultima Necklace of Casting,
                 - Materia: Savage Aim Materia V
                           Amount: 1
        Wrists: Alexandrian Bracelets of Casting,
                 - Materia: Savage Aim Materia V
                           Amount: 1
        Right Ring: Augmented Shire Philosopher's Ring,
                 - Materia: Savage Aim Materia V
                           Amount: 1
        Left Ring: Alexandrian Ring of Casting,
                 - Materia: Quicktongue Materia V
                           Amount: 1
Relic distribution:
        Main Hand - Critical Hit Rate: 120
        Main Hand - Determination: 8
        Main Hand - Spell Speed: 112
Relic stats:
        Main Hand - Critical Hit Rate: 142
        Main Hand - Determination: 9
        Main Hand - Spell Speed: 132
Food:
        Flaugnarde
Allocated stats:
        Intelligence: 35
Result stats with food:
        Magic Damage: 115
        Intelligence: 1604
        Critical Hit Rate: 1195
        Determination: 600
        Accuracy: 593
        Spell Speed: 863
Result stat weight: 3591.475
SS tiers have been taken into account
KaneTW commented 7 years ago

Reopen if the bug is in a current version.