Skip to content

docs: point evaluation.md at the SfM evaluation plan - #40

Merged
hummat merged 1 commit into
mainfrom
docs/sfm-pointer
Aug 24, 2026
Merged

hummat merged 1 commit into
mainfrom
docs/sfm-pointer

Conversation

@hummat

@hummat hummat commented Aug 24, 2026

Copy link
Copy Markdown
Owner

Three-sentence addition to the MUGSQA paragraph, noting that its exact poses make the release usable as a benchmark for our own COLMAP settings, and pointing at #39 for the plan and the coverage table.

Refs: #39

MUGSQA renders with exact poses, which makes its release usable as a
benchmark for COLMAP settings rather than only as prior art on the input
axes. Nothing in this project has varied a matcher or feature setting,
because every comparison was paired on identical SfM output and so cancelled
SfM quality out of the result.

Issue #39 carries the plan and a table of what has been measured against what
has not, so the next person asking does not re-derive it from the git log.

Refs: #39

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 691fd0cf8d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/evaluation.md
Comment on lines +76 to +77
experiment here has varied. Issue #39 holds that plan and the coverage table of
what has and has not been measured.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Link issue 39 explicitly

Use an explicit Markdown link to issue 39 here. In local previews and other renderers that do not apply GitHub's repository-aware issue autolinking, Issue #39 remains plain text, so readers cannot reach the SfM plan or coverage table that this commit is intended to reference.

Useful? React with 👍 / 👎.

@hummat
hummat merged commit f3d69e3 into main Aug 24, 2026
5 checks passed
@hummat
hummat deleted the docs/sfm-pointer branch August 24, 2026 16:13
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.

1 participant