Giter VIP home page Giter VIP logo

pso2act's People

Contributors

alama avatar skrubzer0 avatar sora3100 avatar variantxyz avatar

Stargazers

 avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar

Watchers

 avatar  avatar  avatar  avatar  avatar  avatar

pso2act's Issues

Hero Counters being displayed in a strange way?

Hello. I'm not sure if this is the right place for this, but here we go. I've been using OverParse for a long time, and today wanted to dig deep into my Hero Counter damage for each weapon. But I could only see one "Hero Counter" in my damage logs, instead of the expected "Hero Counter (Sword)", "Hero Counter (TMG)", and "Hero Counter (Talis)".

Digging into the OverParse code, I found that it gets these from the "skills.csv" file, which it pulls from this repository - and digging into that file, it looks like we have the following:

Sword Step Attack (Hero),2448594388,Melee,Hero Sword Step Attack
Hero Counter,2094998893,Melee,Hero Sword Step Attack after dodge (Hero Counter) (Hr skill)
TMG Step Attack (Hero),2834748182,Ranged,Hero TMG Step Attack
TMG Step Attack (Hero),1905792389,Ranged,Hero TMG Step Attack after dodge
Brand New Star,2832005595,PA,Brand New Star
Brand New Star,1399183080,PA,Brand New Star after dodge
Talis Step Attack (Hero),3318755300,Ranged,Hero Talis Step Attack
Talis Step Attack (Hero),1089593935,Ranged,Hero Talis Step Attack after dodge
Talis Step Attack (Hero),4105920492,Ranged,Hero Talis Step Attack after dodge

So, the problems I have here are:

  1. There's an inconsistency between how the counters are displayed.
  2. It's impossible to isolate the damage of TMG counters or Talis counters as they just get merged together with regular step attacks.
  3. It's impossible to isolate the damage of Brand New Star counters, as it just gets merged together with the regular Brand New Star.

I'd like to propose changing the counter versions to "Hero Counter (Sword)", "Hero Counter (TMG)", "Brand New Star (Counter)", and "Hero Counter (Talis)".

Would this be okay? I don't know what the etiquette is here. It seems like an easy enough change for someone to do, or otherwise I could try and submit a pull request. Or would this be unwanted and you prefer it how it is currently for some reason?

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    ๐Ÿ–– Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. ๐Ÿ“Š๐Ÿ“ˆ๐ŸŽ‰

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google โค๏ธ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.