Skip to content

Conversation

@giorgioangel
Copy link
Member

Motivation

  • The transform utility needed to reflect and preserve existing behavior while documenting support for batch processing of TIFXYZ subfolders.
  • A review requested updating comments rather than removing them so the source matches implemented functionality.

Description

  • Added run_tifxyz_batch to apps/src/vc_transform_geom.cpp to traverse an input root, process tifxyz subfolders, and write outputs into a specified output root.
  • Updated main() routing to detect three input modes (single TIFXYZ dir, batch TIFXYZ root dir, and .obj file) and invoke the correct runner.
  • Extended CLI help and unknown-input error text to describe the expanded accepted input types.
  • Updated the vc_transform_geom comment in apps/CMakeLists.txt to match the clarified capabilities while preserving comments in-place.

Testing

  • Verified the source diffs with git diff -- apps/src/vc_transform_geom.cpp apps/CMakeLists.txt and reviewed the modified lines; the diff output matched the intended changes (success).
  • Confirmed file contents and routing logic snippets with nl -ba apps/src/vc_transform_geom.cpp | sed -n '1,20p;248,272p' to validate the new function and comment placement (success).
  • Performed git status --short to ensure a clean working state after committing the changes (success).

Codex Task

@vercel
Copy link

vercel bot commented Feb 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
scrollprize-org Ignored Ignored Preview Feb 12, 2026 8:59pm

Request Review

@giorgioangel giorgioangel merged commit fc85021 into main Feb 12, 2026
8 checks passed
@giorgioangel giorgioangel deleted the codex/create-script-for-processing-tifxyz-meshes branch February 12, 2026 21:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant