Tracker: list migrated files under the series they came from#23
Merged
Conversation
In the per-series manifest, migrated datasets were grouped under a separate data-lectures section, which hid exactly what the table is for: each series' progress (a series could never show green). Migrated files now stay listed in the series they came from — green-highlighted rows sorted to the bottom of each group — with "was: <prior hosting>" on each row. A file consumed by two series (the pandas_panel trio) appears in both groups, marked shared; the tiles still count distinct files. The per-series meters now show real progress: programming 3 of 4 migrated, python.myst 3 of 9, intro 1 of 20. Part of #20. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Updates the migration tracker rendering to keep migrated datasets listed under the lecture series they originated from, improving per-series progress visibility and making shared migrated datasets appear in multiple relevant series sections.
Changes:
- Adds a migrated-row highlight style and sorts per-series tables by migration status (migrated to bottom).
- Reworks
series_manifest()grouping so migrated files are shown under their consuming series (with “was: …” prior-hosting notes). - Updates the per-series meter label and the table lede to reflect the new presentation.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- dataset_status() normalizes a lifecycle `pending` record to the reader-facing `queued` status (with its wave label), so the internal term never reaches the series table, and it sorts with the queued band rather than the unscheduled one (STATUS_RANK covers it as belt and braces). Regression-tested with a synthetic pending record: renders as "queued · wave PX". - series_manifest() docstring and the table lede now say a MIGRATED file consumed by two series appears in both groups — before migration a file has one home, the repo owning the bytes. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #22, from Matt's review of the live tracker.
The per-series manifest grouped migrated datasets under a separate data-lectures section, which hid exactly the story the table exists to tell — a series could never show progress. Migrated files now stay listed in the series they came from, as green-highlighted rows sorted to the bottom of each group, each carrying was: so the where-it-was story is per-file.
The pandas_panel trio was consumed by two series, so those rows appear in both the programming and python.myst groups, marked shared — also listed under …; the stat tiles still count distinct files (noted in the table lede).
The per-series meters now show real progress: programming 3/4 migrated · python.myst 3/9 · intro 1/20.
Verified in both themes; strict build green.
🤖 Generated with Claude Code