Experimental: Show download counts instead of popularity score

Author: szakarias

app/lib/frontend/handlers/experimental.dart

@@ -9,6 +9,7 @@ import '../../shared/cookie_utils.dart';
 const _publicFlags = <String>{
   'dark',
   'search-completion',
+  'download-counts',
 };
 
 const _allFlags = <String>{
@@ -89,6 +90,8 @@ class ExperimentalFlags {
   bool get isDarkModeEnabled => isEnabled('dark');
   bool get isDarkModeDefault => isEnabled('dark-as-default');
 
+  bool get showDownloadCounts => isEnabled('download-counts');
+
   String encodedAsCookie() => _enabled.join(':');
 
   @override

app/lib/frontend/templates/package_misc.dart

@@ -197,6 +197,7 @@ d.Node labeledScoresNodeFromPackageView(PackageView view, {String? version}) {
     likeCount: view.likes,
     grantedPubPoints: view.grantedPubPoints,
     popularity: view.popularity,
+    thirtyDaysDownloads: view.thirtyDaysDownloadCounts,
   );
 }
 

app/lib/frontend/templates/views/pkg/labeled_scores.dart

@@ -2,6 +2,8 @@
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
+import 'package:_pub_shared/format/number_format.dart';
+import 'package:pub_dev/frontend/request_context.dart';
 import 'package:pub_dev/shared/popularity_storage.dart';
 
 import '../../../dom/dom.dart' as d;
@@ -11,6 +13,7 @@ d.Node labeledScoresNode({
   required int likeCount,
   required int? grantedPubPoints,
   required int? popularity,
+  required int? thirtyDaysDownloads,
 }) {
   return d.a(
     classes: ['packages-scores'],
@@ -24,19 +27,32 @@ d.Node labeledScoresNode({
         classes: ['packages-score', 'packages-score-health'],
         child: _labeledScore('pub points', grantedPubPoints, sign: ''),
       ),
-      d.div(
-        classes: ['packages-score', 'packages-score-popularity'],
-        child: _labeledScore(
-          'popularity',
-          popularityStorage.isInvalid ? null : popularity,
-          sign: popularityStorage.isInvalid ? '' : '%',
-        ),
-      ),
+      requestContext.experimentalFlags.showDownloadCounts
+          ? d.div(
+              classes: ['packages-score', 'packages-score-downloads'],
+              child: _labeledScore(
+                'downloads',
+                thirtyDaysDownloads != null
+                    ? computeValueWithSuffix(thirtyDaysDownloads).value
+                    : null,
+                sign: thirtyDaysDownloads != null
+                    ? computeValueWithSuffix(thirtyDaysDownloads).suffix
+                    : '',
+              ),
+            )
+          : d.div(
+              classes: ['packages-score', 'packages-score-popularity'],
+              child: _labeledScore(
+                'popularity',
+                popularityStorage.isInvalid ? null : popularity,
+                sign: popularityStorage.isInvalid ? '' : '%',
+              ),
+            ),
     ],
   );
 }
 
-d.Node _labeledScore(String label, int? value, {required String sign}) {
+d.Node _labeledScore(String label, num? value, {required String sign}) {
   return d.fragment([
     d.div(
       classes: ['packages-score-value', if (value != null) '-has-value'],

app/lib/package/models.dart

@@ -11,6 +11,7 @@ import 'package:_pub_shared/search/tags.dart';
 import 'package:clock/clock.dart';
 import 'package:json_annotation/json_annotation.dart';
 import 'package:pana/models.dart';
+import 'package:pub_dev/service/download_counts/backend.dart';
 import 'package:pub_dev/shared/markdown.dart';
 import 'package:pub_semver/pub_semver.dart';
 
@@ -964,6 +965,7 @@ class PackageView {
 
   final List<String>? topics;
   final int popularity;
+  final int? thirtyDaysDownloadCounts;
 
   PackageView({
     this.screenshots,
@@ -982,6 +984,7 @@ class PackageView {
     this.apiPages,
     this.topics,
     required this.popularity,
+    required this.thirtyDaysDownloadCounts,
   })  : isPending = isPending ?? false,
         tags = tags ?? <String>[];
 
@@ -995,6 +998,7 @@ class PackageView {
     required ScoreCardData scoreCard,
     List<ApiPageRef>? apiPages,
     required int popularity,
+    required int? thirtyDaysDownloadCounts,
   }) {
     final tags = <String>{
       ...package.getTags(),
@@ -1019,6 +1023,7 @@ class PackageView {
       screenshots: scoreCard.panaReport?.screenshots,
       topics: version?.pubspec?.canonicalizedTopics,
       popularity: popularity,
+      thirtyDaysDownloadCounts: thirtyDaysDownloadCounts,
     );
   }
 
@@ -1040,6 +1045,7 @@ class PackageView {
       screenshots: screenshots,
       topics: topics,
       popularity: popularity,
+      thirtyDaysDownloadCounts: thirtyDaysDownloadCounts,
     );
   }
 
@@ -1191,6 +1197,8 @@ class PackagePageData {
       version: version,
       scoreCard: scoreCard,
       popularity: popularityStorage.lookupAsScore(package.name!),
+      thirtyDaysDownloadCounts:
+          downloadCountsBackend.lookup30DayTotalCounts(package.name!),
     );
   }
 }

app/lib/package/models.g.dart

@@ -9,6 +9,7 @@ import 'package:gcloud/service_scope.dart' as ss;
 import 'package:logging/logging.dart';
 import 'package:meta/meta.dart';
 import 'package:pool/pool.dart';
+import 'package:pub_dev/service/download_counts/backend.dart';
 import 'package:pub_dev/shared/exceptions.dart';
 import 'package:pub_dev/shared/popularity_storage.dart';
 import 'package:pub_dev/task/backend.dart';
@@ -97,6 +98,8 @@ class ScoreCardBackend {
         version: pv,
         scoreCard: card,
         popularity: popularityStorage.lookupAsScore(package),
+        thirtyDaysDownloadCounts:
+            downloadCountsBackend.lookup30DayTotalCounts(package),
       );
     });
   }

app/lib/scorecard/backend.dart

@@ -6,15 +6,24 @@
 /// The chunk will have a single digit precision, and will be
 /// rounded down, in order to prevent exaggerated counts.
 String formatWithSuffix(int value) {
+  final f = computeValueWithSuffix(value);
+  return '${_toFixed(f.value)}${f.suffix}';
+}
+
+({num value, String suffix}) computeValueWithSuffix(int value) {
   if (value >= 1000000) {
-    return '${_toFixed(value, 1000000)}m';
+    return (value: _singleDigitDivision(value, 1000000), suffix: 'm');
   } else if (value >= 1000) {
-    return '${_toFixed(value, 1000)}k';
+    return (value: _singleDigitDivision(value, 1000), suffix: 'k');
   } else {
-    return value.toString();
+    return (value: value, suffix: '');
   }
 }
 
-String _toFixed(int value, int d) {
-  return (((value * 10) ~/ d) / 10).toStringAsFixed(1);
+num _singleDigitDivision(int value, int d) {
+  return (((value * 10) ~/ d) / 10);
+}
+
+String _toFixed(num value) {
+  return value < 1000 ? '$value' : value.toStringAsFixed(1);
 }