refactor(blender): stabilize naming synchronization and decouple translator architecture#242
Open
arounamounchili wants to merge 3 commits into
Open
refactor(blender): stabilize naming synchronization and decouple translator architecture#242arounamounchili wants to merge 3 commits into
arounamounchili wants to merge 3 commits into
Conversation
…itecture - Implemented two-tier deferred renaming (timers for GUI, queue for background). - Decoupled platform adapters into a registry-based Translator pattern. - Resolved asynchronous naming inconsistencies and race conditions in CI/CD. - Professionalized error handling and type safety across blender platform. - Maintained 100% test pass rate across 170+ unit and integration tests. Signed-off-by: arounamounchili <patouossa.mounchili@gmail.com>
arounamounchili
changed the title
- Fix NoneType errors in view_layer access during headless tests\n- Stabilize MockCollection (hashability) and MockTimers (re-scheduling) for async operators\n- Cleanup temporary debug prints and optimize imports across the Blender platform\n- Resolve linting issues (duplicated keys, shadowed builtins, contextlib.suppress) Signed-off-by: arounamounchili <patouossa.mounchili@gmail.com>
…tion - Centralize mock_bpy_env.py to tests/ root for unified mocking. - Implement high-fidelity operator mocks (add_empty_link) to ensure scene population. - Fix NoneType errors in physics, sensors, and converter robustness tests. - Remove temporary debug prints and standardize defensive assertions. - Ensure 100% test pass rate in both headless and real Blender environments. Signed-off-by: arounamounchili <patouossa.mounchili@gmail.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.
📝 Description
This PR stabilizes the LinkForge Blender platform by implementing a robust asynchronous naming synchronization system and refactoring the scene translation logic into a decoupled, registry-based architecture.
🖼️ Visual Proof (Recommended)
Tip
For UI changes, 3D viewport features, or URDF exports, verification is much faster with a visual.
Drag and drop screenshots or GIFs here.
🧪 How to Test
💻 Environment
🛠️ Type of change
✅ Checklist
uv run pytestand all tests passuv run pre-commit run --all-filesand all hooks pass