From 84a15d55163dfdb3752f145cbafc8e248e99007c Mon Sep 17 00:00:00 2001 From: didayolo Date: Tue, 17 Feb 2026 19:40:50 +0100 Subject: [PATCH 1/7] Group the submission logs --- .../competitions/detail/submission_modal.tag | 140 ++++++------------ 1 file changed, 45 insertions(+), 95 deletions(-) diff --git a/src/static/riot/competitions/detail/submission_modal.tag b/src/static/riot/competitions/detail/submission_modal.tag index 0961c0b0b..b5c2a29ef 100644 --- a/src/static/riot/competitions/detail/submission_modal.tag +++ b/src/static/riot/competitions/detail/submission_modal.tag @@ -37,96 +37,35 @@
-
-
-
-
- Prediction Logs -
-
- Scoring Logs -
-
-
-
-
-
-
- stdout -
-
- stderr -
-
- Ingestion stdout -
-
- Ingestion stderr -
-
- -
- - 
{ logs.prediction_stdout }
-
- -
- 
{ logs.prediction_stderr }
-
- -
- 
{ logs.prediction_ingestion_stdout }
-
- -
- 
{ logs.prediction_ingestion_stderr }
-
-
-
-
-
- stdout -
-
- stderr -
-
- Ingestion stdout -
-
- Ingestion stderr -
-
- -
- 
{ logs.scoring_stdout }
-
- -
- 
{ logs.scoring_stderr }
-
- -
- 
{ logs.scoring_ingestion_stdout }
-
- -
- 
{ logs.scoring_ingestion_stderr }
-
-
-
+
+
+ Ingestion output +
+
+ Ingestion errors +
+
+ Scoring output +
+
+ Scoring errors +
+
+
+ 
{ showLog(logs.prediction_ingestion_stdout) }
+
+
+ 
{ showLog(logs.prediction_ingestion_stderr) }
+
+
+ 
{ showLog(logs.scoring_stdout) }
+
+
+ 
{ showLog(logs.scoring_stderr) }
@@ -148,6 +87,11 @@ self.leaderboards = [] self.columns = [] + // Check if logs are empty + self.isEmpty = (v) => v == null || (typeof v === "string" && v.trim().length === 0) + self.nonEmpty = (v) => !self.isEmpty(v) + self.showLog = (v) => self.nonEmpty(v) ? v : "No logs for this tab." + self.get_score_details = function (column) { try { let score = _.filter(self.submission.scores, (score) => { @@ -246,10 +190,16 @@ #downloads thead tr th, #downloads tbody tr td font-size 16px !important - .inverted, textarea - color: white - background: #1b1c1d - width: 100% - height: 98% + pre.empty + opacity 0.7 + + .log + color white + background #1b1c1d + + .log textarea + width 100% + height 98% + From 83a1fc110677effbb44ec84f447a3f883d68f06d Mon Sep 17 00:00:00 2001 From: didayolo Date: Wed, 25 Feb 2026 15:55:07 +0100 Subject: [PATCH 2/7] Dynamic tabs instead of always 4 --- .../competitions/detail/submission_modal.tag | 130 ++++++++++++++---- 1 file changed, 100 insertions(+), 30 deletions(-) diff --git a/src/static/riot/competitions/detail/submission_modal.tag b/src/static/riot/competitions/detail/submission_modal.tag index b5c2a29ef..93f9a974a 100644 --- a/src/static/riot/competitions/detail/submission_modal.tag +++ b/src/static/riot/competitions/detail/submission_modal.tag @@ -6,6 +6,7 @@
ADMIN
FACT SHEET ANSWERS
+
@@ -36,50 +37,42 @@
+
-
-
- Ingestion output -
-
- Ingestion errors -
-
- Scoring output -
-
- Scoring errors -
-
-
- 
{ showLog(logs.prediction_ingestion_stdout) }
-
-
- 
{ showLog(logs.prediction_ingestion_stderr) }
+
+
+ {tab.label} +
-
- 
{ showLog(logs.scoring_stdout) }
+ +
+ 
No logs available for this submission.
-
- 
{ showLog(logs.scoring_stderr) }
+ +
+ 
{ showLog(tab.content) }
+
Save
+
+
+ @@ -225,7 +224,7 @@ .file-download margin-top 25px !important - margin-botton 25px !important + margin-bottom 25px !important .graph-frame height 100% From e523ef347f0ced4660776a7f73aeea1e68486864 Mon Sep 17 00:00:00 2001 From: didayolo Date: Wed, 25 Feb 2026 17:47:43 +0100 Subject: [PATCH 6/7] Put back some code that were actually working better --- src/static/riot/competitions/detail/submission_modal.tag | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/static/riot/competitions/detail/submission_modal.tag b/src/static/riot/competitions/detail/submission_modal.tag index 199e98cae..190ce76ed 100644 --- a/src/static/riot/competitions/detail/submission_modal.tag +++ b/src/static/riot/competitions/detail/submission_modal.tag @@ -158,7 +158,8 @@ self.rebuildLogTabs() self.update() setTimeout(() => { - $(self.root).find('.ui.top.attached.menu .item').tab() + //$(self.root).find('.ui.top.attached.menu .item').tab() + $('.ui.top.attached.menu .item').tab() }, 0) }) }) @@ -202,6 +203,11 @@ CODALAB.events.on('submission_clicked', () => { self.submission = opts.submission + // reset per-submission state + self.logs = {} + self.logTabs = [] + self.activeLogTabId = null + // update self.update() self.update_submission_details() let path = self.submission.admin ? 'admin_downloads' : 'downloads' From f3f6154adc0ff5a8a3ab49b9d784f6e325070233 Mon Sep 17 00:00:00 2001 From: didayolo Date: Wed, 25 Feb 2026 18:30:41 +0100 Subject: [PATCH 7/7] Use Semantic UI for inner tabs, clean code --- .../competitions/detail/submission_modal.tag | 61 +++++++++---------- 1 file changed, 30 insertions(+), 31 deletions(-) diff --git a/src/static/riot/competitions/detail/submission_modal.tag b/src/static/riot/competitions/detail/submission_modal.tag index 190ce76ed..04776d01b 100644 --- a/src/static/riot/competitions/detail/submission_modal.tag +++ b/src/static/riot/competitions/detail/submission_modal.tag @@ -40,10 +40,9 @@
-
+
{tab.label}
@@ -52,10 +51,10 @@ 
No logs available for this submission.
-
- 
{ showLog(tab.content) }
+ 
{ show_log(tab.content) }
@@ -82,8 +81,8 @@ self.columns = [] // Logs helpers - self.nonEmpty = (v) => !self.isEmpty(v) - self.showLog = (v) => self.nonEmpty(v) ? self.normalizeLog(v) : "No logs for this tab." + self.non_empty = (v) => !self.is_empty(v) + self.show_log = (v) => self.non_empty(v) ? self.normalizeLog(v) : "No logs for this tab." self.normalizeLog = (v) => { if (v == null) return v if (Array.isArray(v)) return v.join('\n') @@ -92,16 +91,15 @@ } return String(v) } - self.isEmpty = (v) => { + self.is_empty = (v) => { v = self.normalizeLog(v) return v == null || (typeof v === "string" && v.trim().length === 0) } // Dynamic tabs state self.logTabs = [] - self.activeLogTabId = null - self.rebuildLogTabs = () => { + self.rebuild_log_tabs = () => { const prefix = self.submission && self.submission.admin ? 'admin_' : '' const candidates = [ { key:'p_stdout', label:'Prediction output', content: self.logs.prediction_stdout }, @@ -116,16 +114,11 @@ // Keep only non empty tabs self.logTabs = candidates - .filter(t => !self.isEmpty(t.content)) + .filter(t => !self.is_empty(t.content)) .map(t => ({ ...t, fullTabId: `${prefix}${t.key}` })) - // Select a valid tab as active - const stillValid = self.logTabs.find(t => t.fullTabId === self.activeLogTabId) - if (!stillValid) { - self.activeLogTabId = self.logTabs.length ? self.logTabs[0].fullTabId : null - } } self.get_score_details = function (column) { @@ -140,7 +133,7 @@ } self.update_submission_details = () => { self.logs = {} - self.rebuildLogTabs() + self.rebuild_log_tabs() self.update() CODALAB.api.get_submission_details(self.submission.id) .done(function (data) { @@ -151,19 +144,25 @@ self.detailed_result = data.detailed_result self.fact_sheet_answers = data.fact_sheet_answers - _.forEach(data.logs, (item) => { - $.get(item.data_file) - .done(function (content) { - self.logs[item.name] = content - self.rebuildLogTabs() - self.update() - setTimeout(() => { - //$(self.root).find('.ui.top.attached.menu .item').tab() - $('.ui.top.attached.menu .item').tab() - }, 0) - }) + const requests = data.logs.map(item => + $.get(item.data_file) + .done(content => { self.logs[item.name] = content }) + .fail(() => { self.logs[item.name] = "" }) + ) + // When all log files are done loading: + $.when.apply($, requests).always(() => { + self.rebuild_log_tabs() + self.update() + setTimeout(() => { + const $items = $(self.root).find('.ui.top.attached.menu .item') + $items.tab() + // pick the first tab deterministically + if (self.logTabs.length) { + $items.tab('change tab', self.logTabs[0].fullTabId) + } + }, 0) }) - self.rebuildLogTabs() + self.rebuild_log_tabs() self.update() if (self.submission.admin) { _.forEach(data.leaderboards, (leaderboard) => {