Travel Mapping
Web Design Discussion => General Web Design Discussion => Topic started by: bejacob on April 10, 2019, 09:26:49 pm
-
Curious if there is any plan to change the TBD on the ranking of the overall stats part of the page. Not something that matters as I would consider it a low priority. Just wondering.
-
There's been a GitHub issue (https://github.com/TravelMapping/Web/issues/91) for some time now. It pops back up in discussion every now again, but making a fix never seems to gain much traction.
FYI: doing this only from the web side results in massive queries that take awhile to process.
Doesn't seem to me this should matter too much, at least in the case of a "Distance Traveled" ranking; we already use a query to get that very same info on http://travelmapping.net/stat.php
-
I see what Thing342 means now. The query I'm using takes 1.3s, compared with 0.16s for the next slowest I saw, with everything else < 0.1s.
Double that, for one active & one active+preview query, and ouch.
I'll leave this on my own mirror (http://205.209.84.174/) of the site, maybe even make a commit to a separate branch, but not open a pull request just yet.
Eventually I may look into extra tables that can be created during the site update process to speed up the queries.
-
I've got an implementation of the rankings on the user pages installed on the staging site (https://tmstage.teresco.org). It's really slow, but I think it's worth that tradeoff. I have some ideas how to improve efficiency but they likely involve new values being stored in the DB and changes to the site update process to compute and store those values.
Let me know what you think in this thread or in the relevant GitHub issue (https://github.com/TravelMapping/Web/issues/91) and I will decide if and when to get this over on the main site.
-
I think I've got a way to fix the efficiency in the works (and a fix for the incorrect percentages now being shown on tmstage for routes traveled and clinched).
-
A more efficient version has been tested on the development and staging servers, so I am installing on the main site. Let me know if you see any problems with it.
-
A more efficient version has been tested on the development and staging servers, so I am installing on the main site. Let me know if you see any problems with it.
Thanks. I'm currently #3 for routes traveled and routes clinched. mapcat is #1 for both, vespertine #2, oscar #4, charliezeb #5.
HOWEVER: Distance traveled is fukced up now! mapcat is indicated being #21 only! https://travelmapping.net/user/?u=mapcat&units=miles I'm #13 instead of #14.
-
Thanks - I see the problem, hopefully a quick fix this evening.
-
Better?
-
Yep, thanks :)
-
Would it be possible to add the ability to see a list of all users ranked by routes traveled and routes clinched, like we can for distance traveled?
-
Would it be possible to add the ability to see a list of all users ranked by routes traveled and routes clinched, like we can for distance traveled?
Are you thinking of something like this? Update the table on
https://travelmapping.net/stat.php
to have the same information as the table at the bottom of pages like
https://travelmapping.net/user/region.php?units=miles&u=terescoj&rg=NY
where you can reorder the tables by traveled routes or clinched routes.
As you can see, the rank numbers remain tied to the distance traveled but maybe that would be ok for the stat page too as a start.
The DB has some design flaws that make some things, like ranking users by various criteria, inefficient. Some day I'd like to redesign it if I ever learn enough about database design to do it properly.
-
Yes, basically that page, but for traveled routes and clinched routes for all regions, either on the same page or their own page. I think that the rank numbers being tied to the distance traveled would be ok at first.
-
I think the fairly straightforward addition is to add 2 columns to each of the
user.php stat.php page's tables for number of (connected) routes traveled and number of routes clinched.
Added to an open GitHub issue:
https://github.com/TravelMapping/Web/issues/791
-
Are you thinking of something like this? Update the table on
https://travelmapping.net/stat.php
to have the same information as the table at the bottom of pages like
https://travelmapping.net/user/region.php?units=miles&u=terescoj&rg=NY
Seconded.
-
I think the fairly straightforward addition is to add 2 columns to each of the user.php page tables for number of (connected) routes traveled and number of routes clinched.
Nope, that would mean 10 additional columns for by region, and 5 by system!
Today, by region has:
Clinched (miles) + Overall (miles) + % each for active and active+preview systems.
We'd need to add Clinched (routes) + Clinched (%) + Traveled (routes) + Traveled (%) + Overall (routes)
For by system just for active OR active+preview, thus 5 columns.
I think that's too much, isn't it?
-
That would be way too much. I think it would need to be 5 columns in each table:
Rank
Username
Distance (miles/km followed by percent in parens)
Routes traveled (number followed by percent in parens)
Routes clinched (number followed by percent in parens)
It is definitely possible that this is still way too much as long as we will have separate tables for active and active+preview. A compromise might be to have a toggle to show active only or active+preview.
-
That would be way too much. I think it would need to be 5 columns in each table:
Rank
Username
Distance (miles/km followed by percent in parens)
Routes traveled (number followed by percent in parens)
Routes clinched (number followed by percent in parens)
That's https://travelmapping.net/stat.php I seconded. However, I'd like to have separate columns for % so that columns remain sortable.
It is definitely possible that this is still way too much as long as we will have separate tables for active and active+preview. A compromise might be to have a toggle to show active only or active+preview.
I don't like the toggle. Not user-friendly. I usually just look at active+preview, i.e. I had to enable it first.....
-
Sorry, I typed "user.php" thinking "stat.php". I don't propose any changes to user.php.
-
Also, there seems to be an error with the rankings by Routes Clinched. terminal_moraine and formulanone are both tied for 16 but have 1889 and 1888 routes clinched, respectively, and ssoworld and barefoot_driver are both tied for 38 but have 1021 and 1017 routes clinched, respectively.
Edit: This is for active systems only.
-
Also, there seems to be an error with the rankings by Routes Clinched. terminal_moraine and formulanone are both tied for 16 but have 1889 and 1888 routes clinched, respectively, and ssoworld and barefoot_driver are both tied for 38 but have 1021 and 1017 routes clinched, respectively.
Edit: This is for active systems only.
Good catch, thanks for reporting it. Fortunately, it was an easy fix and I think it's good now.