add new functionality to select concurrent nov and varsity outrounds before pairing. Make menial fixes to bugs in outround pairing strategies

Author: JoeyRubas

assets/js/outround.js

@@ -16,40 +16,61 @@ function cycleChoice(event) {
 }
 
 function populateTabCards() {
-  const roundNumber = $("#round-number").data("round-number");
-  if (!roundNumber || !$(".outround-tabcard").length) {
+  const roundNumberEl = $("#round-number");
+  const roundNumbersAttr = roundNumberEl.attr("data-round-numbers");
+  let roundNumbers = [];
+
+  if (roundNumbersAttr) {
+    roundNumbers = roundNumbersAttr
+      .split(",")
+      .map((n) => n.trim())
+      .filter((n) => n);
+  }
+
+  if (!roundNumbers.length) {
+    const fallbackRoundNumber = roundNumberEl.data("round-number");
+    if (fallbackRoundNumber) {
+      roundNumbers = [String(fallbackRoundNumber)];
+    }
+  }
+
+  roundNumbers = [...new Set(roundNumbers)];
+
+  if (!roundNumbers.length || !$(".outround-tabcard").length) {
     return;
   }
-  $.ajax({
-    url: `/outround/${roundNumber}/stats`,
-    success(result) {
-      Object.entries(result).forEach(([teamId, stats]) => {
-        const tabCardElement = $(`.outround-tabcard[team-id=${teamId}]`);
-        const text = [
-          stats.effective_outround_seed,
-          stats.outround_seed,
-          stats.wins,
-          stats.total_speaks.toFixed(2),
-          stats.govs,
-          stats.opps,
-          stats.seed,
-        ].join(" / ");
-        tabCardElement.attr(
-          "title",
-          "Effective Seed / Outround Seed / In-round Wins" +
-            " / Speaks / Govs / Opps / Seed",
-        );
-        tabCardElement.attr("href", `/team/card/${teamId}`);
-        tabCardElement.text(text);
-      });
-    },
+
+  roundNumbers.forEach((roundNumber) => {
+    $.ajax({
+      url: `/outround/${roundNumber}/stats`,
+      success(result) {
+        Object.entries(result).forEach(([teamId, stats]) => {
+          const tabCardElement = $(`.outround-tabcard[team-id=${teamId}]`);
+          const text = [
+            stats.effective_outround_seed,
+            stats.outround_seed,
+            stats.wins,
+            stats.total_speaks.toFixed(2),
+            stats.govs,
+            stats.opps,
+            stats.seed,
+          ].join(" / ");
+          tabCardElement.attr(
+            "title",
+            "Effective Seed / Outround Seed / In-round Wins" +
+              " / Speaks / Govs / Opps / Seed",
+          );
+          tabCardElement.attr("href", `/team/card/${teamId}`);
+          tabCardElement.text(text);
+        });
+      },
+    });
   });
 }
 
 function assignTeam(e) {
   e.preventDefault();
   const teamId = $(e.target).attr("team-id");
-  const oldTeamId = $(e.target).attr("src-team-id");
   const roundId = $(e.target).attr("round-id");
   const position = $(e.target).attr("position");
   const url = `/outround/pairings/assign_team/${roundId}/${position}/${teamId}`;
@@ -67,13 +88,7 @@ function assignTeam(e) {
         $container.find(".team-link").text(result.team.name);
         $container.find(".team-link").attr("href", `/team/${result.team.id}`);
         $container.find(".outround-tabcard").attr("team-id", result.team.id);
-
-        populateTabCards($(`.outround-tabcard[team-id=${result.team.id}]`));
-
-        const $oldTeamTabCard = $(`.outround-tabcard[team-id=${oldTeamId}]`);
-        if ($oldTeamTabCard) {
-          populateTabCards($oldTeamTabCard);
-        }
+        populateTabCards();
       } else {
         window.alert(alertMsg);
       }
@@ -140,7 +155,8 @@ function populateAlternativeJudges() {
   const $parent = $(this).parent();
   const judgeId = $parent.attr("judge-id");
   const roundId = $parent.attr("round-id");
-  const url = `/outround/${roundId}/alternative_judges/${judgeId || ""}`;
+  const query = window.location.search || "";
+  const url = `/outround/${roundId}/alternative_judges/${judgeId || ""}${query}`;
 
   $.ajax({
     url,
@@ -154,29 +170,31 @@ function populateAlternativeJudges() {
 }
 
 function togglePairingRelease(event) {
-  const button = $(".outround-release");
-  const numTeams = button.data("num_teams");
-  const typeOfRound = button.data("type_of_round");
+  const button = $(event.currentTarget);
+  const numTeams = button.attr("data-num_teams");
+  const typeOfRound = button.attr("data-type_of_round");
+  const releaseKey = button.attr("data-release-key");
 
   event.preventDefault();
   $.ajax({
     url: `/outround_pairing/release/${numTeams}/${typeOfRound}`,
     success(result) {
+      const groupSelector = `.outround-release[data-release-key='${releaseKey}']`;
+      const closeBtn = `${groupSelector}.close-pairings-btn`;
+      const openBtn = `${groupSelector}.open-pairings-btn`;
       if (result.pairing_released) {
-        $("#close-pairings").removeClass("d-none");
-        $("#release-pairings").addClass("d-none");
+        $(closeBtn).removeClass("d-none");
+        $(openBtn).addClass("d-none");
       } else {
-        $("#close-pairings").addClass("d-none");
-        $("#release-pairings").removeClass("d-none");
+        $(closeBtn).addClass("d-none");
+        $(openBtn).removeClass("d-none");
       }
     },
   });
 }
 
 $(document).ready(() => {
-  $(".team.outround-tabcard").each((_, element) => {
-    populateTabCards($(element));
-  });
+  populateTabCards();
   $(".choice-update").each((_, element) => {
     $(element).click(cycleChoice);
   });

assets/js/pairing.js

@@ -123,6 +123,11 @@ function assignRoom(e) {
   let $buttonWrapper;
   if (curRoomId) {
     $buttonWrapper = $(`span[round-id=${roundId}][room-id=${curRoomId}]`);
+  } else {
+    $buttonWrapper = $(`.room span[round-id=${roundId}]`).first();
+  }
+  if (!$buttonWrapper.length) {
+    return;
   }
   const $button = $buttonWrapper.find(".btn-sm");
   $button.addClass("disabled");
@@ -148,7 +153,8 @@ function populateAlternativeRooms() {
   const roundId = $parent.attr("round-id");
   const outround = $parent.attr("outround") === "true";
   const baseUrl = outround ? "/outround" : "/round";
-  const url = `${baseUrl}/${roundId}/alternative_rooms/${roomId || ""}`;
+  const query = outround ? window.location.search || "" : "";
+  const url = `${baseUrl}/${roundId}/alternative_rooms/${roomId || ""}${query}`;
 
   $.ajax({
     url,

docs/Advanced-Topics.md

@@ -76,12 +76,12 @@ Below is a reference table of available settings. Most can be modified through t
 +--------------------------+---------------+------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
 | `var_panel_size`         | `3`           | The number of judges on a varsity outs panel.                                                                                                                                                                                                                                                                                                                          |
 +--------------------------+---------------+------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
-| `var_to_nov`             | `2`           | The offset of the novice break to the varsity break.  If novice semis happen when varsity quarters happen, the offset should be 1.  If novice semis happen when varsity octofinals happen, the offset should be 2.                                                                                                                                                     |
-+--------------------------+---------------+------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
 | `var_teams_visible`      | `256`         | The number of teams above which the varsity outround is visible.  For example, if it were 8, quarterfinals and above would be visible, if it were 4, semifinals and above would be visible.                                                                                                                                                                            |
 +--------------------------+---------------+------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
 | `nov_teams_visible`      | `256`         | The number of teams above which the novice outround is visible.  For example, if it were 8, quarterfinals and above would be visible, if it were 4, semifinals and above would be visible.                                                                                                                                                                             |
 +--------------------------+---------------+------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
+| `outround_judge_priority`| `0`           | Controls how judges are prioritized when assigning varsity and novice outrounds together. `0` ("Varsity Wings") assigns varsity first, then novice with remaining judges. `1` ("Novice Chairs") assigns both together by panel slot and gives novice rounds the lower draft position each slot.                                                                         |
++--------------------------+---------------+------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
 | `gov_opp_display`        | `0`           | A toggle to switch the outrounds view from displaying Gov and Opp (when checked) and Team 1 and Team 2 (when value is 2)                                                                                                                                                                                                                                               |
 +--------------------------+---------------+------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
 | `sidelock`               | `0`           | Check this to indicate you are side-locking outrounds. When checked, the gov opps for outrounds will not be random, but will adhere to sidelocks, and will be indicated on the pairing card, and bolded on the front-facing pairing display.                                                                                                                           |

docs/Outrounds.md

@@ -12,11 +12,11 @@ All teams have a `break_preference` field which determines which break they'd pr
 
 It will also perform a number of checks to ensure you have enough rooms and judges.  It does support paneled rounds (as long as they are consistently paneled) -- please consult the `nov_panel_size` and `var_panel_size` TabSettings for more information.  It will let you pair if you don't have enough judges, but it will warn you.
 
-The other tab setting that must be set correctly in order to ensure judges / rooms are not double booked is the `var_to_nov` variable.  If you would like varsity octafinals to happen at the same time as novice quarterfinals, this value should be `2` as the quotient of number of teams in varsity break rounds to the simultaneous novice break round is 2.  If they are happening at the same time, the value should be `1`, if they are octafinals at the same time as semifinals, then it should be 4, etc.
-
 ## Managing Pairings
 
-MIT Tab supports automatic judge assignment for outrounds. On the pairing view, you can click the "Assign Judges" button to automatically assign judges to rounds based on their rankings and scratches (assuming you have set `var_to_nov` correctly - see above - AND all scratches are entered). This will respect panel sizes as configured in `var_panel_size` and `nov_panel_size`.
+MIT Tab supports automatic judge and room assignment for outrounds. On the pairing view, use the Varsity and Novice dropdowns to choose which active outrounds are in scope, then click "Assign Judges" and/or "Assign Rooms". Assignment runs across the selected scope as a single pool, so judges and rooms are not double-booked within the selected rounds.
+
+Automatic judge assignment respects panel sizes as configured in `var_panel_size` and `nov_panel_size`, and can use the `outround_judge_priority` setting when varsity and novice rounds are assigned together.
 
 **Important**: Wing-only judges (judges with the "Wing Only" checkbox enabled) will be automatically excluded from chair assignments. They can still be assigned as panel members but will never be selected as the chair during automatic assignment.
 

mittab/apps/tab/migrations/0036_alter_outround_room_nullable.py

@@ -0,0 +1,25 @@
+# Generated by Django 3.2.25 on 2026-02-24 00:00
+
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("tab", "0035_merge_0032_rankinggroup_0034_add_motion_model"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="outround",
+            name="room",
+            field=models.ForeignKey(
+                blank=True,
+                null=True,
+                on_delete=django.db.models.deletion.CASCADE,
+                related_name="rooms_outrounds",
+                to="tab.room",
+            ),
+        ),
+    ]

mittab/apps/tab/models.py

@@ -491,6 +491,8 @@ class Outround(models.Model):
         (OPP_VIA_FORFEIT, "OPP via Forfeit"),
     )
     room = models.ForeignKey(Room,
+                             blank=True,
+                             null=True,
                              on_delete=models.CASCADE,
                              related_name="rooms_outrounds")
     victor = models.IntegerField(choices=VICTOR_CHOICES, default=0)