Add direct Blender camera framing tool#91
Conversation
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughA new Changesframe_camera_to_targets full-stack feature
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested labels
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint install failed: dependency version conflict. Check your lock file or package.json. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
frame_camera_to_targetsto both ViperMesh Blender addon copies.Validation
npx tsx scripts/test/test-blender-camera-framing-tool.tspython -m py_compile desktop/assets/vipermesh-addon.py public/downloads/vipermesh-addon.pynpx tsx scripts/test/test-tool-guide-trigger-coverage.tsnpx tsx scripts/test/test-blender-capability-inventory.tsnpx tsc --noEmit --incremental falsenpm run lint(passes; existing baseline-browser-mapping age warning)git diff --check(passes; Windows line-ending warnings only)Summary by CodeRabbit
New Features
Documentation