ppy / osu-web

the browser-facing portion of osu!
https://osu.ppy.sh
GNU Affero General Public License v3.0
973 stars 380 forks source link

Home page user count should include lazer users #10816

Closed peppy closed 7 months ago

peppy commented 9 months ago

Currently, it's sourcing from an osu_counts row that bancho is responsible for.

Instead, it should add up all users across all active builds (allow_bancho = 1) and probably cache once a minute or so. Probably quite a simple one:

root@db-master:osu> select sum(users) from osu_builds where allow_bancho = 1;
+------------+
| sum(users) |
+------------+
| 8323       |
+------------+
nanaya commented 9 months ago

The one using osu_counts table looks like for a different total users. It's the one being incremented on user registration. Do you mean the current online users count from osu_banchostats table?

peppy commented 7 months ago

Tracking at #10991 with more up-to-date information.