Skip to content

Add preview renders and upload artifacts#16

Open
justinleeyang wants to merge 2 commits intoaukilabs:developfrom
justinleeyang:bounty/preview-outputs
Open

Add preview renders and upload artifacts#16
justinleeyang wants to merge 2 commits intoaukilabs:developfrom
justinleeyang:bounty/preview-outputs

Conversation

@justinleeyang
Copy link
Copy Markdown

@justinleeyang justinleeyang commented Mar 15, 2026

Summary

  • Render top-down + angled preview images and an orbit preview video after export (best-effort; does not fail the job).
  • Emit camera path JSONs and document preview outputs.
  • Upload preview artifacts from the runner with dedicated data types.
  • Add plyfile dependency for PLY bounds.

Testing

  • python -m py_compile run.py
  • cargo fmt --all
  • cargo test --workspace --all-features
  • cargo clippy --workspace --all-targets --all-features -- -D warnings

Notes: preview video rendering requires fmpeg and nerfstudio runtime; not exercised locally.

@justinleeyang
Copy link
Copy Markdown
Author

Quick follow-up: this branch is ready for review. I also ran the local checks successfully:\n\n- python -m py_compile run.py\n- cargo fmt --all\n- cargo test --workspace --all-features\n- cargo clippy --workspace --all-targets --all-features -- -D warnings\n\nThe preview rendering remains best-effort, so the main splat pipeline still succeeds even if preview generation is unavailable in a given runtime. Happy to adjust naming or artifact types if you want a different convention.

@justinleeyang justinleeyang force-pushed the bounty/preview-outputs branch from a66a8db to d2ff79f Compare March 28, 2026 00:03
@justinleeyang
Copy link
Copy Markdown
Author

I rebased this branch on the latest develop and resolved the README conflict. The PR should be mergeable now. Let me know if you prefer the output artifacts list in a different doc location.

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