Skip to content
This repository has been archived by the owner on Sep 12, 2024. It is now read-only.

Playerstats correct playtime #56

Open
Nepmia opened this issue Nov 29, 2020 · 6 comments
Open

Playerstats correct playtime #56

Nepmia opened this issue Nov 29, 2020 · 6 comments

Comments

@Nepmia
Copy link
Member

Nepmia commented Nov 29, 2020

playerstatistic have meta, meta include total playtime.

But the numbed given by the API is not correct, seems the normal number got divided by 5. sadly we don't have the exact number so we can't really process this info correctly.

@TrapinchO
Copy link

Actually the number is 4.7 btw

@Nepmia
Copy link
Member Author

Nepmia commented Nov 29, 2020

yes but WHY it is divided by 4.7

@TrapinchO
Copy link

That is the question we all ask. Perhaps it shows exact playtime, but in strange unit

@dukio
Copy link

dukio commented Nov 29, 2020

I don't know if it's true, but I think someone once explained that wynn caches every 5 minute (more or less) the players online, and to determine the playtime it checks how many times that player was online in the cached online players..... and the Api returns that number instead of the number*5.

Anyway changing the returned data now will break most if not all the apps using the playtime from stats route....so it's probably best to leave it as it is.

Why 4.7 instead of 5 I have no idea, personally on wynndata I use 5 and it seems to work fine.

@TrapinchO
Copy link

Interesting theory.

In my testing and testing of the author of Wynndata 4.7 proved to be the best.

Check #discussions in the dev discord to see yourself

@MikhaD
Copy link

MikhaD commented Jul 22, 2021

Instead of changing the returned data and risking breaking things the documentation for the API should at least be updated to reflect the fact that it is minutes online / 4.7 instead of just minutes online.

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

No branches or pull requests

4 participants