Skip to content

Fix review markers, Rhiza link and PDF metadata - #3

Merged
tschm merged 1 commit into
mainfrom
fix-vision-review
Aug 5, 2026
Merged

Fix review markers, Rhiza link and PDF metadata#3
tschm merged 1 commit into
mainfrom
fix-vision-review

Conversation

@tschm

@tschm tschm commented Aug 5, 2026

Copy link
Copy Markdown
Member

Three mechanical fixes to vision.tex. No prose changes — every sentence is untouched.

1. The red review markers ship in the published PDF

3e8e898 ("Mark today's additions in red") wrapped three passages in \color{red}:

  • the prior-art footnote on SecDB / Beacon / Deltix
  • the reconciliation-in-banks passage in A New Direction
  • the whole Build or Buy section

That works as a marker for a reviewer who knows the convention, but the CI job publishes vision.pdf to gh-pages on every push to main, so a reader arriving at the public URL sees red text with no way to interpret it as anything other than emphasis or an unfinished draft. Build or Buy is one of the strongest sections in the document to have flagged that way. This restores all three to plain body text.

If the intent was to keep the marking for a while longer, the alternative is a line under the byline explaining the convention — happy to do that instead.

2. The Rhiza link pointed at the curriculum

In Keeping the Platform Consistent, "Rhiza was built on a different premise: scaffolding is not a one-time generation but a continuous synchronisation" linked to Jebel-Quant/rhiza-education. A reader following the argument about continuous sync landed on lesson material. Now points at Jebel-Quant/rhiza.

3. The PDF had no title or author metadata

\author{} is empty and there is no \title, so the distributed PDF carried no document title — it presents as vision.pdf in browser tabs, reference managers and anywhere it gets forwarded. Adds pdftitle and pdfauthor to the existing \hypersetup block. Nothing renders differently on the page.

Verification

Built locally with the same command and flags CI uses (xelatex -halt-on-error -interaction=nonstopmode, twice): compiles clean, 14 pages, unchanged from before. Confirmed no \color{red} and no rhiza-education remain, and that the built PDF's Info dictionary now carries Title = A Technology Vision for Quantitative Trading and Author = Thomas Schmelzer.

🤖 Generated with Claude Code

The red colouring added in 3e8e898 marked the day's additions for
review, but it ships in the published PDF, where a reader has no way to
read it as anything but emphasis or an unfinished draft. The three
passages (the prior-art footnote, the reconciliation passage and the
Build or Buy section) become plain body text.

The "Rhiza was built on a different premise" link pointed at
rhiza-education, the curriculum, rather than at the tool itself.

Set pdftitle and pdfauthor so the distributed PDF carries a title and an
author rather than showing up as vision.pdf with no metadata.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 5, 2026 11:15

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR makes three mechanical LaTeX updates to vision.tex to improve the published PDF’s presentation and correctness without changing the document’s prose.

Changes:

  • Adds pdftitle and pdfauthor to the existing \hypersetup block so the generated PDF includes title/author metadata.
  • Removes \color{red} review-marker wrapping from the prior-art footnote and two marked sections so the public PDF no longer renders red “draft” text.
  • Fixes the Rhiza hyperlink to point to Jebel-Quant/rhiza instead of Jebel-Quant/rhiza-education.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tschm
tschm merged commit 787f1cc into main Aug 5, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants