Define segmentation when it is first mentioned#364
Define segmentation when it is first mentioned#364K-Meech merged 3 commits intodatacarpentry:mainfrom
Conversation
Thank you!Thank you for your pull request 😃 🤖 This automated message can help you check the rendered files in your submission for clarity. If you have any questions, please feel free to open an issue in {sandpaper}. If you have files that automatically render output (e.g. R Markdown), then you should check for the following:
Rendered Changes🔍 Inspect the changes: https://github.com/datacarpentry/image-processing/compare/md-outputs..md-outputs-PR-364 The following changes were observed in the rendered markdown documents: What does this mean?If you have source files that require output and figures to be generated (e.g. R Markdown), then it is important to make sure the generated figures and output are reproducible. This output provides a way for you to inspect the output in a diff-friendly manner so that it's easy to see the changes that occur due to new software versions or randomisation. ⏱️ Updated at 2026-04-27 22:25:44 +0000 |
K-Meech
left a comment
There was a problem hiding this comment.
Thanks for the contribution @jni ! I agree - a clearer definition of segmentation would be great to start this lesson.
I've made some very minor suggestions on wording below. @marcodallavecchia / @uschille - would be good if you could also take a quick look.
| where we change the pixels of an image to make the image easier to analyze. | ||
| which is the partitioning of an image into disjoint regions, or *segments*. | ||
| The different segments can then be analyzed separately. | ||
| In thresholding, we convert an image from colour or grayscale into |
There was a problem hiding this comment.
Probably good to add a blank line above 'In thresholding...' to start a new paragraph. Makes this section easier to read in the rendered docs.
|
Great! @jni - are you happy to make the small changes suggested above? Then we can get this merged 😄 |
Uses simpler language for learners. Co-authored-by: Kimberly Meechan <24316371+K-Meech@users.noreply.github.com>
Start a new paragraph when talking about thresholding. Co-authored-by: Ulf Schiller <uschille@users.noreply.github.com>
|
Thanks all! Yes, good suggestions. It seems this is ready to go then! 🙏 |
Auto-generated via `{sandpaper}`
Source : 53ec159
Branch : md-outputs
Author : GitHub Actions <actions@github.com>
Time : 2026-04-28 08:35:57 +0000
Message : markdown source builds
Auto-generated via `{sandpaper}`
Source : 90b5bfb
Branch : main
Author : Kimberly Meechan <24316371+K-Meech@users.noreply.github.com>
Time : 2026-04-28 08:33:56 +0000
Message : Merge pull request #364 from jni/define-segmentation
Define segmentation when it is first mentioned
While reading this lesson, I noticed that segmentation was used but not clearly defined.
I think a simple definition will help learners understand what they are working towards.