Skip to content

Overview: freshness badge — orange past a week, red past two #24

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mmcky merged 1 commit into from
Jul 17, 2026
Merged

Overview: freshness badge — orange past a week, red past two #24

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 28 additions & 0 deletions scripts/render_audit.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -207,6 +207,13 @@
.meter .seg.b { background: var(--meter-b); }
.meter.mini { height: 8px; max-width: 340px; margin: 6px 0 2px; border-radius: 4px; }
tr.row-migrated td { background: var(--ok-bg); }
.freshness {
display: inline-block; border-radius: 6px; padding: 4px 12px; margin-top: 12px;
font-size: 13px; font-weight: 600;
background: var(--ok-bg); color: var(--ok); border: 1px solid var(--line);
}
.freshness.warn { background: var(--warn-bg); color: var(--warn); }
.freshness.crit { background: var(--crit-bg); color: var(--crit); }
/* bar list — one measure, single hue, labeled rows */
.barlist { display: grid; grid-template-columns: max-content 1fr max-content; gap: 6px 12px; align-items: center; margin: 14px 0; }
.barlist .lbl { font-size: 13px; }
Expand Down Expand Up @@ -460,6 +467,27 @@ def render_index(audit: dict) -> str:
<a href="{DL}/blob/main/scripts/build_audit.py">a script</a> — not maintained by hand.
Companion to the migration of lecture data into
<a href="{DL}">QuantEcon/data-lectures</a>.</p>
<span id="freshness" class="freshness" data-generated="{esc(audit["generated"])}"
title="Rebuilt on every change to data-lectures and weekly on schedule — a badge older than a week means the scheduled rebuild has stopped.">
● generated {esc(audit["generated"])}</span>
<script>
(function () {{
var el = document.getElementById('freshness');
var gen = new Date(el.dataset.generated + 'T00:00:00Z');
var days = Math.floor((Date.now() - gen.getTime()) / 86400000);
if (isNaN(days)) return;
var age = days <= 0 ? 'today' : days === 1 ? 'yesterday' : days + ' days ago';
if (days > 14) {{
el.classList.add('crit');
el.textContent = '✗ stale — last updated ' + age + ' (' + el.dataset.generated + '); the scheduled rebuild has stopped';
}} else if (days > 7) {{
el.classList.add('warn');
el.textContent = '⚠ last updated ' + age + ' (' + el.dataset.generated + ') — the weekly rebuild is overdue';
}} else {{
el.textContent = '● up to date — last updated ' + age + ' (' + el.dataset.generated + ')';
}}
Comment on lines +476 to +488

@mmcky mmcky Jul 17, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Addressed in follow-up PR #25 — as a documented push-back rather than the suggested change. The run stamp is date-only (midnight UTC) and the weekly cron fires Mondays at 05:17 UTC, so comparing raw milliseconds against the 7-day threshold would flip the badge orange every Monday between 00:00 and the cron completing — a built-in weekly false alarm. Flooring to full calendar days is deliberate: orange means a full scheduled run has been missed, which is the signal the badge exists to give. The reasoning is now a code comment beside the calculation so it survives future reviews.

}})();
</script>
</header>
{tiles}
{migration_meter(audit)}
Expand Down
Loading