Backend: Add show_scores_on_leaderboard field to ChallengePhaseSplit model and update related components (#4985)

* Backend: Add show_scores_on_leaderboard field to ChallengePhaseSplit model and update related components

* Frontend: Add unit test to ensure chosenMetrics is set to empty when show_scores_on_leaderboard is false

Author: RishabhJain2018

apps/challenges/admin.py

@@ -264,6 +264,7 @@ class ChallengePhaseSplitAdmin(ImportExportTimeStampedAdmin):
         "leaderboard_decimal_precision",
         "is_leaderboard_order_descending",
         "show_execution_time",
+        "show_scores_on_leaderboard",
     )
     list_filter = ("visibility",)
     search_fields = (

apps/challenges/migrations/0113_add_github_branch_field_and_unique_constraint.py

@@ -1,6 +1,6 @@
 # Generated by Django 2.2.20 on 2025-07-09 15:00
 
-from django.db import migrations
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
@@ -10,23 +10,37 @@ class Migration(migrations.Migration):
     ]
 
     operations = [
-        migrations.RunSQL(
-            sql=(
-                "DO $$\n"
-                "BEGIN\n"
-                "    IF NOT EXISTS (\n"
-                "        SELECT 1 FROM information_schema.columns\n"
-                "        WHERE table_name='challenge'\n"
-                "        AND column_name='github_branch'\n"
-                "    ) THEN\n"
-                "        ALTER TABLE challenge ADD COLUMN github_branch "
-                "varchar(200) NULL DEFAULT '';\n"
-                "    END IF;\n"
-                "END$$;"
-            ),
-            reverse_sql=(
-                "ALTER TABLE challenge DROP COLUMN IF EXISTS github_branch;"
-            ),
+        migrations.SeparateDatabaseAndState(
+            state_operations=[
+                migrations.AddField(
+                    model_name="challenge",
+                    name="github_branch",
+                    field=models.CharField(
+                        blank=True, default="", max_length=200, null=True
+                    ),
+                ),
+            ],
+            database_operations=[
+                migrations.RunSQL(
+                    sql=(
+                        "DO $$\n"
+                        "BEGIN\n"
+                        "    IF NOT EXISTS (\n"
+                        "        SELECT 1 FROM information_schema.columns\n"
+                        "        WHERE table_name='challenge'\n"
+                        "        AND column_name='github_branch'\n"
+                        "    ) THEN\n"
+                        "        ALTER TABLE challenge ADD COLUMN github_branch "
+                        "varchar(200) NULL DEFAULT '';\n"
+                        "    END IF;\n"
+                        "END$$;"
+                    ),
+                    reverse_sql=(
+                        "ALTER TABLE challenge DROP COLUMN IF EXISTS "
+                        "github_branch;"
+                    ),
+                ),
+            ],
         ),
         # Add a partial unique constraint
         # Only applies when both fields are not empty

apps/challenges/migrations/0115_add_show_scores_on_leaderboard_to_challenge_phase_split.py

@@ -0,0 +1,18 @@
+# Generated by Django 2.2.20 on 2026-02-05 17:56
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("challenges", "0114_add_require_complete_profile"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="challengephasesplit",
+            name="show_scores_on_leaderboard",
+            field=models.BooleanField(default=True),
+        ),
+    ]

apps/challenges/models.py

@@ -502,6 +502,7 @@ class ChallengePhaseSplit(TimeStampedModel):
     is_leaderboard_order_descending = models.BooleanField(default=True)
     show_leaderboard_by_latest_submission = models.BooleanField(default=False)
     show_execution_time = models.BooleanField(default=False)
+    show_scores_on_leaderboard = models.BooleanField(default=True)
     # Allow ordering leaderboard by all metrics
     is_multi_metric_leaderboard = models.BooleanField(default=True)
 

apps/challenges/serializers.py

@@ -190,6 +190,7 @@ class Meta:
             "visibility",
             "show_leaderboard_by_latest_submission",
             "show_execution_time",
+            "show_scores_on_leaderboard",
             "leaderboard_schema",
             "is_multi_metric_leaderboard",
         )
@@ -384,6 +385,7 @@ class Meta:
             "is_leaderboard_order_descending",
             "show_leaderboard_by_latest_submission",
             "show_execution_time",
+            "show_scores_on_leaderboard",
         )
 
 

apps/jobs/utils.py

@@ -514,6 +514,7 @@ def calculate_distinct_sorted_leaderboard_data(
             team_list.append(data["submission__participant_team__team_name"])
 
     leaderboard_labels = challenge_phase_split.leaderboard.schema["labels"]
+    show_scores = challenge_phase_split.show_scores_on_leaderboard
     for item in distinct_sorted_leaderboard_data:
         item_result = []
         for index in leaderboard_labels:
@@ -529,6 +530,12 @@ def calculate_distinct_sorted_leaderboard_data(
                 item["error"]["error_{0}".format(index)]
                 for index in leaderboard_labels
             ]
+
+        if not show_scores:
+            item["result"] = []
+            item["error"] = None
+            item.pop("filtering_score", None)
+            item.pop("filtering_error", None)
     return distinct_sorted_leaderboard_data, status.HTTP_200_OK
 
 

frontend/src/js/controllers/challengeCtrl.js

@@ -1071,6 +1071,9 @@
                     vm.selectedPhaseSplit = response.data;
                     vm.sortLeaderboardTextOption = (vm.selectedPhaseSplit.show_leaderboard_by_latest_submission) ?
                         "Sort by best":"Sort by latest";
+                    if (vm.selectedPhaseSplit.show_scores_on_leaderboard === false) {
+                        vm.chosenMetrics = [];
+                    }
                 },
                 onError: function (response) {
                     var error = response.data;
@@ -1105,7 +1108,11 @@
 
                         var leaderboardLabels = vm.leaderboard[i].leaderboard__schema.labels;
                         var index = leaderboardLabels.findIndex(label => label === vm.orderLeaderboardBy);
-                        vm.chosenMetrics = index !== -1 ? [index.toString()]: undefined;
+                        if (vm.selectedPhaseSplit && vm.selectedPhaseSplit.show_scores_on_leaderboard === false) {
+                            vm.chosenMetrics = [];
+                        } else {
+                            vm.chosenMetrics = index !== -1 ? [index.toString()]: undefined;
+                        }
                         vm.leaderboard[i]['submission__submitted_at_formatted'] = vm.leaderboard[i]['submission__submitted_at'];
                         vm.initial_ranking[vm.leaderboard[i].id] = i+1;
                         var dateTimeNow = moment(new Date());
@@ -3108,7 +3115,9 @@
         };
 
         vm.openLeaderboardDropdown = function() {
-            if (vm.chosenMetrics == undefined) {
+            if (vm.chosenMetrics == undefined &&
+                (!vm.selectedPhaseSplit || vm.selectedPhaseSplit.show_scores_on_leaderboard !== false) &&
+                vm.leaderboard && vm.leaderboard[0]) {
                 var index = [];
                 for (var k = 0; k < vm.leaderboard[0].leaderboard__schema.labels.length; k++) {
                     var label = vm.leaderboard[0].leaderboard__schema.labels[k].toString().trim();

frontend/src/views/web/challenge/leaderboard.html

@@ -40,7 +40,7 @@ <h5 class="w-300">Leaderboard</h5>
                 </span>
             </div>
         </div>
-        <div class="row" ng-if="challenge.isMultiMetricLeaderboardEnabled[challenge.phaseSplitId]">
+        <div class="row" ng-if="challenge.isMultiMetricLeaderboardEnabled[challenge.phaseSplitId] && (!challenge.selectedPhaseSplit || challenge.selectedPhaseSplit.show_scores_on_leaderboard !== false)">
             <div class="col xs12 s6">
                 <span>
                     <md-select ng-model="challenge.orderLeaderboardBy" placeholder="Order by metric" class="rm-margin">
@@ -74,7 +74,7 @@ <h5 class="w-300">Leaderboard</h5>
                             <label for="getAllEntries"></label>
                             <span  class="fs-16 w-300 complete-leaderboard"> {{ challenge.getAllEntriesTestOption }}</span>
                         </div>
-                        <md-select name="leaderboarddropDownmenu" class="right" ng-model="challenge.chosenMetrics" placeholder="Visible Metrics" class="rm-margin" md-on-open="challenge.openLeaderboardDropdown()" md-on-close="challenge.openLeaderboardDropdown()" multiple>
+                        <md-select name="leaderboarddropDownmenu" class="right" ng-model="challenge.chosenMetrics" placeholder="Visible Metrics" class="rm-margin" md-on-open="challenge.openLeaderboardDropdown()" md-on-close="challenge.openLeaderboardDropdown()" multiple ng-if="!challenge.selectedPhaseSplit || challenge.selectedPhaseSplit.show_scores_on_leaderboard !== false">
                             <md-option value="{{i}}" ng-repeat="(i, key) in challenge.leaderboard[0].leaderboard__schema.labels">
                                 <span ng-if="!challenge.leaderboardDropdown" class="fs-16 w-300">Visible Metrics</span>
                                 <span ng-if="challenge.leaderboardDropdown" class="fs-16 w-300">{{key}}</span>
@@ -107,7 +107,7 @@ <h5 class="w-300">Leaderboard</h5>
                                     </span>
                                 </a>
                             </td>
-                            <td ng-repeat="(i, key) in challenge.leaderboard[0].leaderboard__schema.labels track by $index" ng-if="challenge.chosenMetrics.indexOf($index.toString()) !== -1">
+                            <td ng-repeat="(i, key) in challenge.leaderboard[0].leaderboard__schema.labels track by $index" ng-if="(challenge.chosenMetrics || []).indexOf($index.toString()) !== -1">
                                 <a href="#"
                                     ng-click="$parent.challenge.sortLeaderboard($parent.challenge, 'number', $index);">
                                     <span class="w-300 fs-20 leaderboard-label" ng-if="challenge.isMetricOrderedAscending(key) && key != challenge.orderLeaderboardBy ">{{key}} (&#x2193;) <span class="description" ng-if="challenge.getLabelDescription(key).length != 0">{{challenge.getLabelDescription(key)}}</span></span>
@@ -168,7 +168,7 @@ <h5 class="w-300">Leaderboard</h5>
                                     id="verified-badge" class="new badge green-background baseline-tag" data-badge-caption="V"
                                     ng-if="key.submission__is_verified_by_host"></span>
                             </td>
-                            <td ng-repeat="(i, score) in key.result track by $index" ng-if="challenge.chosenMetrics.indexOf($index.toString()) !== -1">
+                            <td ng-repeat="(i, score) in key.result track by $index" ng-if="(challenge.chosenMetrics || []).indexOf($index.toString()) !== -1">
                                 <span class="w-400 fs-16" ng-if="challenge.leaderboard[0].leaderboard__schema.labels[i] == challenge.orderLeaderboardBy">{{score | number : challenge.selectedPhaseSplit.leaderboard_decimal_precision}}</span>
                                 <span class="w-300 fs-16" ng-if="challenge.leaderboard[0].leaderboard__schema.labels[i] != challenge.orderLeaderboardBy">{{score | number : challenge.selectedPhaseSplit.leaderboard_decimal_precision}}</span>
                                 <span class="new badge orange-background fs-16 w-300 partial-evaluation"

frontend/tests/controllers-test/challengeCtrl.test.js

@@ -3085,6 +3085,32 @@ describe('Unit tests for challenge controller', function () {
             expect(vm.showSubmissionMetaAttributesOnLeaderboard).toBe(false);
             expect(vm.leaderboard[0].chosenMetrics).toBeUndefined();
         });
+
+        it('should set chosenMetrics to empty when show_scores_on_leaderboard is false', function () {
+            var fakeLeaderboard = [{
+                id: 1,
+                leaderboard__schema: { labels: ['accuracy', 'loss'] },
+                orderLeaderboardBy: 'accuracy',
+                submission__submission_metadata: null,
+                submission__submitted_at: new Date(Date.now() - 2 * 365 * 24 * 60 * 60 * 1000).toISOString()
+            }];
+            spyOn(utilities, 'sendRequest').and.callFake(function (params) {
+                if (params.url && params.url.includes('/leaderboard/')) {
+                    params.callback.onSuccess({ data: { results: fakeLeaderboard } });
+                } else if (params.url && params.url.includes('/challenge_phase_split/')) {
+                    params.callback.onSuccess({
+                        data: {
+                            show_scores_on_leaderboard: false,
+                            show_leaderboard_by_latest_submission: false
+                        }
+                    });
+                }
+            });
+
+            vm.getLeaderboard(123);
+
+            expect(vm.chosenMetrics).toEqual([]);
+        });
     });
 
     describe('Unit tests for re-run submission logic', function () {
@@ -3869,6 +3895,14 @@ describe('Unit tests for challenge controller', function () {
             vm.openLeaderboardDropdown();
             expect(vm.leaderboardDropdown).toBe(false);
         });
+
+        it('should not populate chosenMetrics when show_scores_on_leaderboard is false', function () {
+            vm.chosenMetrics = undefined;
+            vm.selectedPhaseSplit = { show_scores_on_leaderboard: false };
+            vm.openLeaderboardDropdown();
+            expect(vm.chosenMetrics).toBeUndefined();
+            expect(vm.leaderboardDropdown).toBe(true);
+        });
     });
 
     describe('Unit tests for getTrophySize function', function () {

tests/unit/challenges/test_models.py

@@ -231,6 +231,17 @@ def test__str__(self):
             string_to_compare, self.challenge_phase_split.__str__()
         )
 
+    def test_show_scores_on_leaderboard_default(self):
+        """Test that show_scores_on_leaderboard defaults to True."""
+        self.assertTrue(self.challenge_phase_split.show_scores_on_leaderboard)
+
+    def test_show_scores_on_leaderboard_can_be_false(self):
+        """Test that show_scores_on_leaderboard can be set to False."""
+        self.challenge_phase_split.show_scores_on_leaderboard = False
+        self.challenge_phase_split.save()
+        self.challenge_phase_split.refresh_from_db()
+        self.assertFalse(self.challenge_phase_split.show_scores_on_leaderboard)
+
 
 class LeaderboardDataTestCase(BaseTestCase):
     def setUp(self):