Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SMN.lua #49

Closed
Crevox opened this issue Jul 4, 2014 · 3 comments
Closed

SMN.lua #49

Crevox opened this issue Jul 4, 2014 · 3 comments

Comments

@Crevox
Copy link

Crevox commented Jul 4, 2014

  1. Line 58: Ramuh's Astral Flow is spelled "Judgement Bolt" which is incorrect, causing no gearswap to occur. The proper spelling is "Judgment Bolt."

  2. Mana Cede and Elemental Siphon sets do not function. They must be added as midcasts. This is one example of how it's done to better understand this (added in job_midcast, preferably moved over to proper midcast system):

if (spell.english=='Mana Cede') then
        equip(sets.precast.JA['Mana Cede'])
    elseif (spell.english=='Elemental Siphon')  then
        equip(sets.precast.JA['Elemental Siphon'])
end
  1. The "siphon system" is not working properly. As an example of a bad scenario, In the event of 2 dark weathers on a lightsday, a light spirit is used, which offers greatly reduced MP compared to a dark spirit.
@Crevox
Copy link
Author

Crevox commented Jul 19, 2014

Updated with another issue.

@Kinematics
Copy link
Owner

First, sorry for not noticing this issue. No one's used my issue tracker in so long, forgot it existed.

  1. Fixed.
  2. Rename the sets as sets.midcast['Mana Cede'] and sets.midcast['Elemental Siphon'] to force those sets to be used during midcast. You shouldn't have to modify the rules. I'll need to do testing on my own for how they work.
  3. Fixed.

@Kinematics
Copy link
Owner

Closing this since the bugs were fixed. The potential remaining issue is a logic one, not a code one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants