Skip to content

Dev - #22

Merged
scheuerdo merged 3 commits into
mainfrom
dev
Aug 1, 2026
Merged

Dev#22
scheuerdo merged 3 commits into
mainfrom
dev

Conversation

@scheuerdo

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings August 1, 2026 08:16
@scheuerdo
scheuerdo merged commit 8bbe532 into main Aug 1, 2026
1 check passed

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

Updates the poster “pipeline + priors” banner layout so prior-panel typography and sizing are better suited for poster viewing distance, including title/caption formatting and a configurable title font size.

Changes:

  • Increased prior panel width and adjusted vertical layout constants to accommodate larger type.
  • Reflowed prior captions (and added title wrapping) for better fit at larger caption sizes.
  • Added --title-size CLI option and threaded title_size through to prior_graphs.draw_panel, plus increased legend font size.

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

Comment thread poster/pipeline_priors.py
Comment on lines +131 to +142
# Wrap long titles at the first "(" so they fit in their panel width.
# Re-wrap captions: the hard-coded \n breaks were tuned for the smaller standalone
# caption size; re-flow to ~38 chars per line for the larger size used here.
panels = [
(
key,
title.replace(" (", "\n("),
spec,
textwrap.fill(cap.replace("\n", " "), width=38),
)
for key, title, spec, cap in panels
]
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.

3 participants