Skip to content

Describe segment profiles sampled end point to end point - #440

Merged
PierreRaybaut merged 1 commit into
developfrom
fix/segment-profile-sampling
Oct 26, 2026
Merged

PierreRaybaut merged 1 commit into
developfrom
fix/segment-profile-sampling

Conversation

@PierreRaybaut

Copy link
Copy Markdown
Contributor

Companion of DataLab-Platform/Sigima#102 (closes DataLab-Platform/Sigima#101), same branch name.

Summary

Sigima now samples segment profiles at regular steps from their start point to their end point included, with one sample per pixel along the axis where the segment spans the most pixels (as in Sigima 1.3). Since DataLab-Platform/Sigima#98, they were sampled in steps of the smallest distance between pixel centers, which could omit the end point (e.g. the last pixel of a diagonal) and allocate huge arrays for images with a tiny pixel spacing.

  • doc/features/image/menu_analysis.rst: description of the segment profile updated.
  • French translation updated (checked with a gettext build + sphinx_intl update: no change).

No code change: the profile dialog and parameters are unchanged.

Validation

  • Against Sigima fix/segment-profile-sampling: profile_app_test.py, provenance_processing_unit_test.py, provenance_capture_unit_test.py pass (39).

Segment profiles are now sampled at regular steps from their start point to
their end point included, with one sample per pixel along the axis where the
segment spans the most pixels (DataLab-Platform/Sigima#101): they were
sampled in steps of the smallest distance between pixel centers, which could
omit the end point and allocate huge arrays.

Assisted-by: Claude Opus 5.5
@PierreRaybaut PierreRaybaut added the documentation Improvements or additions to documentation label Oct 26, 2026
@PierreRaybaut
PierreRaybaut merged commit ca433fc into develop Oct 26, 2026
6 checks passed
@PierreRaybaut
PierreRaybaut deleted the fix/segment-profile-sampling branch October 26, 2026 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant