-
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Best Div "time ago" not updating #72
Comments
For reference, in the main cup, I had won div 22 a while ago, then at a later date improved my best to win div 20 and the time listed under "best div rank" updated to reflect that (even though it was still 1st), so I think all these "time ago" texts should be consistent with any improvement. |
Good catch, there's a mismatch between the best division stat and the rank that gets displayed there. Should be a simple fix. Regarding whether these stats should update if the same is achieved again later, I would say the most intuitive way for these stats to work is to show the first instance that rank/div was achieved, no matter if there is a more recent instance or one in a higher div (in the case of div ranks). Right now that's how it works for Best rank and Best div, but Best div rank specifically shows the best division rank in the highest div (so if you got 1st in div 22 and later get 1st in div 20, it'll update to that div 20 win). This logic was originally implemented by @codecat, maybe she can add some context to that decision (and why the other stats don't update when the same is achieved again). Whatever the decision is, I'll probably add a clarification note to the tooltips either way so we don't need to guess what the stats really mean. |
The idea was that it would show the highest rank in any division, not the best division. For example, at the time of writing, my profile says my best division was 2 (placed 11th 3 years ago), but my best div rank was 6th (in div 11, 2 years ago). The logic is a bit confusing I must admit. I'm open to changing it. |
Just to clarify, I meant Best rank in div shows the best rank in the best div you got it in (not just the best div overall of course, that's what Best div is for). So if you had gotten 6th in a lower div, it would show that instead of the first time you got 6th. |
These changes have been merged and deployed, your reported issues should be fixed (along with some extra tooltips that should better explain the Best div rank stat). Feel free to take a look and confirm it yourself @ezio416 in case I missed something. |
@davidbmaier A few things seem broken. Now it's showing that my placement of 18th in div 2 is my best (which is from the first time I got div 2 in reruns - far from my best), and in the rerun panel it's showing I got 1st of div 22 (when I've won div 4, and div 22 obviously isn't a rerun). It's also showing the div 22 win under main cups as being my best when I've won div 20 (22 was my first win). It seems like it's now just taking the very first result that matches the div/div rank. Is this expected? edit - yeah based on the new tooltips this is expected, but I'm kinda sad it doesn't actually show my best results anymore (i.e. my rerun div 4 and main div 20 wins no longer showing) |
I'd say the current logic is more consistent, but I get your point. The "Best div rank" stat isn't really meant to show the best overall rank (or highest div you got the best div rank in), it's just supposed to show what your highest rank within a div was (which obviously becomes a bit pointless once you have won a div). Additionally I'm thinking a switch to change between "first instance" and "most recent instance" could be cool for these stats (since duplicates can clearly occur). But that probably needs some more thinking, especially if we do actually hide some of that data. I'll reopen this issue for now (since we already have some discussion in here) but make it an enhancement item instead of a bug - and I'll hopefully find some time to look into this soon. |
I've added both the latest and the first instance to the stats, maybe that's a good first compromise. Would be interested to hear what you guys think and if this solves the main issues, or if there's still some missing data you'd like to see in this context. |
In COTD reruns, I've gotten div 2 before, but today I got it again and improved my best placement in it from 7th to 2nd. My new best placement was updated, but the time it says I got my best div still says it was 5 months ago, implying that I got 2nd place at that time.
https://trackmania.io/#/player/ezio/cotd
The text was updated successfully, but these errors were encountered: