fix: driver_namespace id_type and SDF-grid meshing in skills#9
Merged
Conversation
The drivers skill's driver_namespace worked example assigned var.targets[0].id = bpy.context.scene without first setting id_type='SCENE'. A SINGLE_PROP target id pointer defaults to OBJECT, so the bare assignment raises TypeError: DriverTarget.id expected a Object type, not Scene on both 4.5.10 LTS and 5.1.1. Added the id_type line (matching snippet driver-with-custom-function.py) and clarified the SINGLE_PROP prose. Verified: driver now attaches and location.z evaluates to [0.0, 2.425, 4.9985] across frames 1/50/100 on both builds. Signed-off-by: fOuttaMyPaint <TMhospitalitystrategies@gmail.com>
build_remesh_via_sdf wired MeshToSDFGrid's 'SDF Grid' output (a grid/float socket) into VolumeToMesh's 'Volume' input (a geometry socket). That link is is_valid=False, so VolumeToMesh received no data and the evaluated modifier produced 0 vertices on both 4.5.10 LTS and 5.1.1, at every Voxel Size/Band Width/Threshold combination tried. Root cause: VolumeToMesh meshes a volume geometry (as produced by MeshToVolume), not a bare grid. Switched to GeometryNodeGridToMesh (Grid input is type-compatible) with threshold=0.0 (SDF zero-level). Verified: cube remesh now yields 10088 verts (5.1.1) / 9602 (4.5.10), zrange (-1.0, 1.0). Updated the heading and node-reference table accordingly. Signed-off-by: fOuttaMyPaint <TMhospitalitystrategies@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
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
Two in-Blender findings from executing the content on 4.5.10 LTS and 5.1.1, both reproduced and both fixes verified on both builds.
F1 — driver_namespace example raised TypeError (fix)
skills/drivers-and-app-handlers/SKILL.md: the worked example didvar.targets[0].id = bpy.context.scenewithout first settingid_type='SCENE'. ASINGLE_PROPtarget id pointer defaults toOBJECT, so the assignment raisedTypeError: DriverTarget.id expected a Object type, not Sceneon both versions.RAISES: TypeError: ... DriverTarget.id expected a Object type, not Scenevar.targets[0].id_type = 'SCENE'before theid =line (matching the already-correct snippetdriver-with-custom-function.pyL28); also clarified theSINGLE_PROPprose.location.zevaluates to[0.0, 2.425, 4.9985]across frames 1/50/100.F2 — SDF-remesh example produced 0 vertices (diagnosed, then fixed)
skills/geometry-nodes-python/SKILL.md:build_remesh_via_sdfwiredMeshToSDFGrid→VolumeToMesh.MeshToSDFGrid.outputs["SDF Grid"]is a grid/float socket (type=VALUE,NodeSocketFloat);VolumeToMesh.inputs["Volume"]isNodeSocketGeometry. The link isis_valid=False— incompatible socket types — so VolumeToMesh gets no data and emits 0 verts at every Voxel Size / Band Width / Threshold combination on both builds. Control:MeshToVolume → VolumeToMeshisis_valid=Trueand meshes fine (8216/7778 verts), confirming VolumeToMesh is for volume geometry, not bare grids.GeometryNodeGridToMesh("Grid to Mesh") —IN=['Grid','Threshold','Adaptivity'] OUT=['Mesh'],Gridinput type-compatible with the SDF grid output.GridToMesh, wireSDF Grid → Grid, defaultthreshold=0.0(SDF zero-level); updated heading + node-reference table.zrange (-1.0, 1.0)(the cube spans ±1).Constraints
Counts unchanged (12/6/2/17). No edits to CHANGELOG / CLAUDE
**Version:**/ ROADMAP**Current:**. standards-version stays 1.10.0. CLAUDE.md untouched (no context-mode block). Both commits signed (-s). Per known release.yml behavior thisfix:will cut a patch.🤖 Generated with Claude Code