Add uv support, CUDA 12 compatibility, and improved Linux setup#1294
Open
MasahiroOgawa wants to merge 7 commits into
Open
Add uv support, CUDA 12 compatibility, and improved Linux setup#1294MasahiroOgawa wants to merge 7 commits into
MasahiroOgawa wants to merge 7 commits into
Conversation
Update submodule to include fixes for: - embree4 support (with embree3 fallback) - FFmpeg 5+ API compatibility - CUDA 12 compilation fixes - VideoUtils.hpp bug fix 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Add Ubuntu 24.04 to tested platforms - Add uv sync as recommended setup method - Keep Conda as alternative option - Update SIBR viewer build to use Ninja by default - Add note about avoiding Anaconda in PATH on Ubuntu 24.04 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Add instructions for running viewer after building from source - Add PRIME offload workaround for hybrid GPU laptops (NVIDIA Optimus) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Document the required step of installing diff-gaussian-rasterization, simple-knn, and fused-ssim submodules after environment setup, since these CUDA extensions require compilation and cannot be installed via uv sync or conda alone. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Clarify that after building from source, the SIBR install directory is SIBR_viewers/install with executables in SIBR_viewers/install/bin/. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Fix network_gui.py socket read to handle TCP fragmentation properly - Add SO_REUSEADDR to prevent "Address already in use" errors - Use UTF-8 encoding for dataset paths (supports non-ASCII characters) - Update README with clear instructions for live training visualization - Document that -s dataset path is required for the network viewer 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Changes
uv Support
pyproject.tomlwith PyTorch CUDA 12.1 wheel configurationSIBR Viewers
-s <dataset path>is required for network viewerNetwork GUI Fixes
SO_REUSEADDRto prevent "Address already in use" errorsTest plan
🤖 Generated with Claude Code
Test result
Now it successfully displays Gaussians using SIBR remote viewer during training like this;
