gw2raidar icon indicating copy to clipboard operation
gw2raidar copied to clipboard

Arty Fail When should be success

Open merforga opened this issue 7 years ago • 6 comments
trafficstars

There's a few of these coming through, I'll try to catch them as I see them but this one should be success. Looks like not detecting anything past Phase 1: https://www.gw2raidar.com/encounter/CirculationDonationsRadiosFlicksBang https://uat.gw2raidar.com/encounter/SessionsAdmissionsShaftDebatePresented

(have set it to success for time being)

merforga avatar Mar 27 '18 00:03 merforga

https://www.gw2raidar.com/encounter/EliteBiasDigitalVillagesRangers 20180616-222700.evtc.zip

https://www.gw2raidar.com/encounter/ConspiraciesFlourishedKneeledLiabilitiesUsers 20180517-105740.evtc.zip

These were success (showing up on leaderboards) when they're actually failed. All others which i've removed from arty leaderboard seem to follow the same things:

  1. All wipe on last phase (ie it shows a short phase 3)
  2. Not everyone actually died but considering its a wipe, everyone should have died which means we're missing the last death in the parse somehow
  3. The health update on Arty which should technically tell us Arty is still alive seems to not be triggering so it's not flagging as fail.

Any ideas?

merforga avatar Jun 19 '18 06:06 merforga

I've uploaded changes that should prevent this, but the side effect I can't easily avoid is that arty and arkk logs will not detect as success after the first run per day (unless we somehow finally sort out the instid issues)

Toeofdoom avatar Jun 19 '18 11:06 Toeofdoom

Sorry couldn't chat before.

Hmmm that's not ideal for the hc community since they'll be running multiple times a day =x. Is it possible to do a total damage check instead? It would then limit it to the <1% wipes which i think will be an even smaller sample size.

I know it's a pain for just a few encounters but it would be more robust imo.

merforga avatar Jun 19 '18 11:06 merforga

It already checked that the boss was below 3% health, but that check gets confused by the instid stuff I mentioned unfortunately

Toeofdoom avatar Jun 19 '18 14:06 Toeofdoom

Not checking the boss health via event status but actually looking at total damage dealt by the group. If it's significantly less than the boss HP then it's most definitely a fail.

merforga avatar Jun 19 '18 22:06 merforga

@Toeofdoom do you think you'll have time to look at this this weekend or soonish? Basically change the current implementation you did of checking reward box to checking total damage done against Arty's hp (lower margin for error imo)

merforga avatar Jul 06 '18 01:07 merforga