You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
-**Clarity:** Comprehensive but very long (~19KB); the 8-geometry-type taxonomy is buried in dense prose.
280
+
-**Completeness:** Excellent — covers all 8 geometry types with examples, including a complete worked example for importing from text files. Possibly too much for a single filter doc.
-**Notes:** "DREAM3D Review (Geometry)" group is unusual — verify this is intended. Typo "dimenionality" appears multiple times. Each geometry-type section duplicates content from per-type filters. Consider promoting the geometry-type taxonomy into a shared concept page and trimming this doc to filter-specific guidance.
-**Completeness:** Has duplicate transformation-type listing (bullets at lines 96-103 and the same content as a table at lines 107-114). One needs to go.
291
+
-**Accessibility:** Node vs Image distinction is critical and well-explained; row-major matrix encoding explained.
292
+
-**Figures Needed:** Already has many. Could add a diagram explaining the four-element axis-angle representation.
293
+
-**Real-World Viz:** Already excellent.
294
+
-**Units Clarity:** Rotation angle units (degrees) explicit; translation in geometry's coordinate units; scale dimensionless.
-**Notes:** Remove the duplicated transformation-type listing. Convert "[Combine Transformation Matrices](CombineTransformationMatricesFilter.md)" link to a proper MyST link (already is). Add Required Input Sources. The relative `./Filter.md` should be checked — current uses both `./` and bare forms.
297
+
298
+
-[ ]**RotateSampleRefFrameFilter** (SimplnxCore)
299
+
-**Clarity:** Verified-only-for-axis-aligned-90/180 warning is at the top — good. Distinction from ApplyTransformation (sample reference frame vs geometric rotation) is not made explicit.
300
+
-**Completeness:** Has rotation matrix equivalent shown. Note about origin shift after rotation is valuable.
301
+
-**Accessibility:** "Sample reference frame" jargon assumed; the term needs first-use definition.
302
+
-**Figures Needed:** Diagram showing sample frame rotation (the axes of the sample relabeled) vs geometric rotation (the data physically rotated).
303
+
-**Real-World Viz:** Already has EBSD rotation example.
304
+
-**Units Clarity:** Rotation angle in degrees; axis is dimensionless unit vector.
-**Notes:** Add explicit "When to Use vs ApplyTransformation" guidance. Make verified-axis-only banner more prominent. Cross-link to SetImageGeomOriginScaling for the origin reset workflow. Add Required Input Sources.
307
+
308
+
-[ ]**ResampleImageGeomFilter** (SimplnxCore)
309
+
-**Clarity:** Three resampling modes (Spacing, Scaling, Exact Dimensions) each with worked numerical examples. Clear.
310
+
-**Completeness:** "No interpolation, closest cell wins" behavior noted. Renumber Features + NeighborList removal warning present.
311
+
-**Accessibility:** Spacing/dimensions/scaling distinction depends on understanding Image Geometry.
312
+
-**Figures Needed:** Before/after voxel-grid diagram showing how a 2x spacing reduction halves the cell count.
313
+
-**Real-World Viz:** Cubic Small IN100 sliced at different resolutions side-by-side.
314
+
-**Units Clarity:** Spacing values are in **physical units** (same as input geometry's spacing); scaling is **percent**; exact dimensions are in **cells**.
315
+
-**Concept Links:** spacing vs scaling vs dimensions, closest-cell resampling, Feature renumbering
316
+
-**Notes:** Cross-link to ApplyTransformation is present but uses `./` path; convert to plain MyST form. Add explicit units callouts per mode. Add Required Input Sources.
-**Clarity:** Brief, just describes the "last one wins" rule.
320
+
-**Completeness:** Missing: when to use this, how to specify the target image geometry's spacing/dimensions, what happens when target is finer than source.
321
+
-**Accessibility:** Rectilinear Grid concept not introduced.
322
+
-**Figures Needed:** Side-by-side diagram of variable-spacing RectGrid → regular Image Geom with the "last one wins" cells highlighted.
323
+
-**Real-World Viz:** A real RectGrid resampled to a uniform Image Geom.
324
+
-**Units Clarity:** Target spacing in physical units.
325
+
-**Concept Links:** Rectilinear Grid vs Image Geometry, downsampling vs upsampling
326
+
-**Notes:** Add "When to Use" section. Explain why "last one wins" was chosen (interpolation is wrong for label data). Add Required Input Sources.
327
+
328
+
-[ ]**PadImageGeometryFilter** (SimplnxCore)
329
+
-**Clarity:** Very brief; the figures are helpful but the prose underdocuments the "default padding value" semantics.
330
+
-**Completeness:** Doesn't explain what the *Update Origin* option does or when to use it.
331
+
-**Accessibility:** "Default padding value" is undefined.
332
+
-**Figures Needed:** Already has before/after. Could add a 3-panel diagram showing Update Origin ON vs OFF.
333
+
-**Real-World Viz:** Before/after with both Update Origin settings.
334
+
-**Units Clarity:** Pad amounts are in **cells/voxels**. Default value is in whatever units the target array uses.
-**Notes:** Expand "default padding value" to "Each padded cell is initialized to the user-specified Default Value, which is interpreted in the same units as each cell-level array being padded". Document *Update Origin* and *Update Spacing* options. Add Required Input Sources.
337
+
338
+
-[ ]**QuickSurfaceMeshFilter** (SimplnxCore)
339
+
-**Clarity:** Deprecation notice at top recommending "Surface Nets". Node Types table is excellent. Triangle-pair-per-cell-face algorithm explained.
340
+
-**Completeness:** Has good images for each Node Type. The Verify Triangle Winding reference is implicit; needs a link.
341
+
-**Accessibility:** "Stair stepped" surface mesh result not pictured directly.
342
+
-**Figures Needed:** Already has plenty. Could use a small diagram of how a single voxel face becomes 2 triangles.
-**Notes:** Improve deprecation banner — make clear when to use this vs Surface Nets. Convert "see Verify Triangle Winding documentation" to a MyST link. Add Required Input Sources (FeatureIds from a segment filter).
-**Clarity:** Three modes (Manual, Random, Random With Range) are listed but the "subvolume" parameter is undefined.
349
+
-**Completeness:** Missing: how is the subvolume specified (min/max cell indices? physical bounds?). What data types support each random mode? What about boolean arrays?
350
+
-**Accessibility:** Reader has to guess the subvolume specification UI.
351
+
-**Figures Needed:** Diagram showing a subvolume highlighted within a larger volume; before/after.
352
+
-**Real-World Viz:** Before/after on a real cube showing the initialized subvolume.
353
+
-**Units Clarity:** Subvolume bounds in **cells/voxels** vs **physical units** — must be stated.
354
+
-**Concept Links:** subvolume, cell-level initialization, random number generation seed
355
+
-**Notes:** Document the subvolume specification mode and units. Note random-mode behavior for boolean, integer, and float types separately. Document the random seed parameter. Add Required Input Sources.
-**Notes:** Move deprecation notice into a clear callout. Fix "dimenionality" typo. Cross-link to CreateGeometry filter. Add Required Input Sources note (none — geometry is created from user parameters).
-**Clarity:** Brief and adequate. Output format documented.
381
+
-**Completeness:** Missing motivating context for why you'd combine matrices (the answer is in ApplyTransformation — referenced indirectly there).
382
+
-**Accessibility:** Assumes user knows row-major 4x4 convention.
383
+
-**Figures Needed:** Optional — a diagram showing two 4x4 matrices multiplying into a single 4x4 result.
384
+
-**Real-World Viz:** N/A.
385
+
-**Units Clarity:** N/A (matrix is dimensionless apart from translation components).
386
+
-**Concept Links:** matrix composition, transformation order, row-major matrix order
387
+
-**Notes:** Add a short "Why Use This Filter" pointing to ApplyTransformationToGeometry's caveat about successive image-geometry transformations. Note that matrices are applied left-to-right in the order listed. Add Required Input Sources.
-**Notes:** Document both Origin and Spacing parameter sets. Document the linkable parameters (only-change-origin, only-change-spacing options). Expand the "Put Origin at Center" explanation. Add Required Input Sources.
398
+
399
+
-[ ]**CropImageGeometryFilter** (SimplnxCore)
400
+
-**Clarity:** Three worked examples with images. Inclusive-bounds note is critical and called out.
401
+
-**Completeness:** Voxels-vs-physical-coordinates mode is implicit; the "Use Physical Bounds" parameter should be stated explicitly.
402
+
-**Accessibility:** Voxel index 0-based convention is implied via examples.
403
+
-**Figures Needed:** Already adequate.
404
+
-**Real-World Viz:** Already present.
405
+
-**Units Clarity:** Bounds are in **cells (0-based, inclusive)** OR **physical units (depending on parameter)** — explicit per-mode.
406
+
-**Concept Links:** ROI cropping, inclusive bounds, voxel indexing
407
+
-**Notes:** Add explicit "Bounds Mode" subsection naming the parameter that toggles voxels vs physical. State units per mode. Update NeighborList warning to use the standard pattern shared with RequireMinimumSizeFeatures.
408
+
409
+
-[ ]**AppendImageGeometryFilter** (SimplnxCore)
410
+
-**Clarity:** Exhaustive examples for X/Y/Z directions with figures.
411
+
-**Completeness:** "Resolution" terminology used in one place — should be "Spacing" per the modern simplnx convention.
412
+
-**Accessibility:** Good.
413
+
-**Figures Needed:** Already comprehensive.
414
+
-**Real-World Viz:** Already present.
415
+
-**Units Clarity:** Append happens in cell-count units along the chosen direction; the *Check Spacing* option checks physical-unit match.
-**Notes:** Replace remaining "Resolution" with "Spacing" for consistency. Cross-link to CreateImageGeometry for the geometry-definition concepts. Add Required Input Sources.
418
+
419
+
-[ ]**PartitionGeometryFilter** (SimplnxCore)
420
+
-**Clarity:** Four modes with excellent visual examples and walk-throughs.
421
+
-**Completeness:** "Reconstruction (Reconstruction)" group/subgroup is a stutter — verify.
422
+
-**Accessibility:** Long doc but well-organized with figures.
423
+
-**Figures Needed:** Already excellent.
424
+
-**Real-World Viz:** Already present.
425
+
-**Units Clarity:** Cell Length is in physical units; Number of Cells Per Axis is in **integer cell counts**; Origin in physical units; Min/Max in physical units.
-**Clarity:** "Implicit vs explicit" framing is jargon; the example output is clear.
431
+
-**Completeness:** Three output modes documented with raster scheme.
432
+
-**Accessibility:** Reader benefits from concrete example, which is provided.
433
+
-**Figures Needed:** Optional — a small voxel-grid diagram with one cell highlighted showing its (i,j,k) index and (x,y,z) physical position.
434
+
-**Real-World Viz:** N/A — output is numerical.
435
+
-**Units Clarity:** Physical coordinates in geometry's physical units; indices are **integer cell indices (0-based)**.
436
+
-**Concept Links:** cell indexing scheme, physical coordinates, ijk vs xyz, raster order
437
+
-**Notes:** Rewrite "implicit/explicit" framing in plain language ("makes the per-cell coordinates available as a regular cell-level array"). Add a "When to Use" — typically for CSV/text export workflows. Add Required Input Sources (Image Geometry).
438
+
439
+
---
440
+
441
+
## Batches 6-9: Not Yet Triaged
257
442
258
443
The following batches have been identified in the design spec but have not yet been triaged. Each batch will be triaged at the start of its work cycle.
259
444
@@ -442,10 +627,6 @@ The following batches have been identified in the design spec but have not yet b
0 commit comments