Skip to content

Commit 2f8b521

Browse files
Donglai Weiclaude
andcommitted
NISB BANIS: turn off per-crop relabel_connected_components
base_banis.yaml inherited relabel_connected_components: true, which renumbered label IDs per crop but did NOT renumber label_aux. The runtime skeleton_aware_edt path then did `skel_crop == instance_id` with mismatched IDs, silently falling back to a plain boundary EDT — disabling the skeleton signal that v2+/erosion2 set up. Flip the default off in base_banis.yaml so v0/v1 still get stable IDs and v2+ actually use their precomputed seg_skeleton caches. The schema default (False) already matched this intent; only the explicit override here was forcing the bug. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent ebdcec4 commit 2f8b521

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

tutorials/neuron_nisb/base_banis.yaml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,10 @@ default:
4848
# - decode_affinity_cc expects the first 3 channels in axis0, axis1, axis2 order
4949
label_transform:
5050
erosion: 0
51-
relabel_connected_components: true
51+
# Keep label IDs stable across crops so precomputed label_aux
52+
# (e.g. seg_skeleton for skeleton_aware_edt in v2+) matches the
53+
# runtime label per-instance.
54+
relabel_connected_components: false
5255
relabel_connectivity: 6
5356
targets:
5457
- name: affinity

0 commit comments

Comments
 (0)