search

nuclearfurnace / rs-storm-parser

A Blizzard StormReplay parser in Rust.
1 stars 0 forks source link

What's left? #2

Open arranf opened 6 years ago

arranf commented 6 years ago

Hey!

I was just about to start writing a parser in Rust then stumbled across yours!

Could you give me a general update on the state of the project and what's left to do to get it to feature parity with other parsers?

Arran

tobz commented 6 years ago

This is an incredibly late response, but... I need to take a look at this again.

My general recollection is that most of the "basics" are done... the primary events, etc. Beyond that, there needs to be support to parse the lobby archive so that we can extract BattleTags.

Beyond that, most of the work would be deriving statistics. This is doing the work like parsing KDAs, damage done, doing something with the events we have. I kind of waffled back and forth on whether we should care about this. Technically a caller could parse the JSON output and do it themselves if we gave them all of the events... but the amount of data is pretty large (although that's largely in part due to it being JSON)

arranf commented 6 years ago

Do you plan on continuing to work on this? If so I'd be happy to collaborate!

Arran

On Wed, 2 May 2018 at 17:47 Toby Lawrence notifications@github.com wrote:

This is an incredibly late response, but... I need to take a look at this again.

My general recollection was that most of the "basics" are done... the primary events, etc. Beyond that, there needs to be support to parse the lobby archive so that we can extract BattleTags.

Beyond that, most of the work would be deriving statistics. This is doing the work like parsing KDAs, damage done, doing something with the events we have. I kind of waffled back and forth on whether we should care about this. Technically a caller could parse the JSON output and do it themselves if we gave them all of the events... but the amount of data is pretty large (although that's largely in part due to it being JSON)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/nuclearfurnace/rs-storm-parser/issues/2#issuecomment-386044253, or mute the thread https://github.com/notifications/unsubscribe-auth/AITM9lejpj7egeLm68f41Y9qUyJz-BFWks5tueMZgaJpZM4SEUA4 .