Feat/youtube v2 support apple silicon#326
Open
vedanthvdev wants to merge 2 commits intogoogle:masterfrom
Open
Conversation
Major modernization update: - Updated to Python 3.8+ (removed Python 2 support) - Optimized for macOS Apple Silicon (M1, M2, M3, M4) - Added Retina display support with proper DPI scaling - Improved GUI with native macOS appearance Core Changes: - spatialmedia/gui.py: Removed Python 2 compatibility, added macOS optimizations - spatialmedia/__main__.py: Updated shebang to python3 - setup.py: Migrated from distutils to setuptools with modern configuration - requirements.txt: Updated dependencies with version constraints New Files: - SETUP_MAC_M_SERIES.md: Comprehensive setup guide for Apple Silicon Macs - build_macos_app.sh: Automated build script for macOS app bundles Documentation: - README.md: Updated with Python 3 and M series information - Added quick start guides and troubleshooting sections Technical Improvements: - Platform-specific scaling (1.4x for macOS, 2.0x for others) - Native macOS themed toplevel windows - Fixed string encoding for Python 3 - Enhanced error handling and console output Compatibility: - Python 3.8, 3.9, 3.10, 3.11, 3.12 - macOS 10.13+ (Intel and Apple Silicon) - Windows 10, 11 - Linux (modern distributions) Tested on: macOS Sequoia with Apple Silicon
- Add YouTube V2 metadata injection (SV3D, PRHD, EQUI boxes) - Fix Box.print_box() method for file opening in GUI - Update to Python 3.8+ with Apple Silicon support - Configure GUI for macOS with proper tkinter settings - Update all dependencies to modern versions Core Changes: - spatialmedia/mpeg/box.py: Add missing print_box() method - spatialmedia/gui.py: Add V2 metadata injection, macOS optimizations - spatialmedia/__init__.py: Add version 2.1.0 - spatialmedia/__main__.py: Add --version flag - setup.py: Migrate to setuptools, Python 3.8+ requirement - .python-version: Set to 3.12.8 for pyenv Bug Fixes: - Fix AttributeError when opening files in GUI - Fix GUI metadata injection to include V2 for YouTube Tested: - All 8 original tests passing - V2 metadata verified (SV3D, PRHD, EQUI boxes present) - YouTube compatibility confirmed - GUI and CLI both functional
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
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.
No description provided.