Add label mesh rendering pipeline - #3
Open
davidackerman wants to merge 20 commits into
Open
Conversation
This reverts commit c765e79.
Label meshing (mesh_from_labels): - Read + mesh the ROI block-by-block (parallel) and weld seams, so a sparse but huge bbox doesn't materialize the whole box at once. Auto-engages by planned read size vs. a memory threshold that respects the job's cgroup limit (LSF/containers), not just the node's free RAM. - target_vertices caps what we LOAD (planner never meshes finer than the budget); decimate_fraction then reduces below it, per segment so instance colors survive. fast-simplification added as the decimation backend. - Removed the confusing simplify_factor knob entirely (smooth + decimate only). Per-keyframe 3D background: - Capture Neuroglancer projectionBackgroundColor per keyframe on bake/update, interpolate it across transitions, and apply/animate it per frame in both the PNG render and the .blend export. - Propagate it across keyframes via the "Update current frame" prompt, mirroring the change back into each keyframe's stored ng_state so the NG link round-trips. UI: render controls regrouped into labeled rows (movie/render/mesh/scene) and the timeline bar into move/keyframe/output; "quality"->"detail"; decimate dropdown; opacity slider relabeled (all layers). README + tests updated.
Per-keyframe `waypoint` flag: the camera passes through the keyframe at speed instead of easing to a stop (and any hold is ignored), so a run of waypoints reads as one continuous flow. Easing is derived per-segment from endpoint type — stop->stop ease-in-out, stop->wp ease-in, wp->stop ease-out, wp->wp linear — which also generalizes the director's smooth-ends curve. Timelines with no waypoints are unchanged. Toggle lives in the transition-edit popover; persisted via DurationReq.
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.
@stuarteberg
Summary
Testing
conda run -n cinemap python -m pytest tests(54 passed, 49 warnings)Notes