Skip to content
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

Top 10 can show NaN #1228

Open
reedy opened this issue Jul 8, 2023 · 2 comments
Open

Top 10 can show NaN #1228

reedy opened this issue Jul 8, 2023 · 2 comments
Labels

Comments

@reedy
Copy link
Collaborator

reedy commented Jul 8, 2023

Screenshot 2023-07-08 at 12 05 10

Noticed when reviewing #1222, but not caused by it... Presumably ue to import issues (#1197) with my current test dataset, the plane rows are messed up.

But this can also be seen on a new account (well, one with no flights) on the live site too:

Screenshot 2023-07-08 at 12 07 37

This can definitely be handled much more nicely

@reedy reedy added the bug label Jul 8, 2023
@kil0ba
Copy link

kil0ba commented Jul 9, 2023

I also faced this issue but only with top 10 by mileage, top 10 by segments works as espected.

I found it in production in my account

@reedy
Copy link
Collaborator Author

reedy commented Jul 10, 2023

It's a bit better after destroying and recreating the database... But there's clearly some bad handling in here

Screenshot 2023-07-11 at 00 22 31

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

No branches or pull requests

2 participants