Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Horizontal segments #1201

Merged
merged 5 commits into from
Jul 17, 2024
Merged

Horizontal segments #1201

merged 5 commits into from
Jul 17, 2024

Conversation

rmburg
Copy link
Contributor

@rmburg rmburg commented Jul 15, 2024

Why? What?

What the title says.
Also extends the segmenter panel to show horizontal segments.

Based on #1179.

ToDo / Known Issues

  • It's perfect

Ideas for Next Iterations (Not This PR)

  • Image segmenter clean-up
  • Line detection
  • Other algorithms
    • Goal posts
    • Penalty spots

How to Test

Use the new "Direction" toggle in the image segmenter twix panel

@rmburg rmburg enabled auto-merge July 15, 2024 14:43
@schluis schluis assigned schluis and unassigned schluis Jul 15, 2024
@schluis schluis added the is:Dependent This is blocked because something else needs to happen first. label Jul 15, 2024
@rmburg rmburg force-pushed the horizontal-segments branch from bfb801a to cc3216b Compare July 16, 2024 07:26
@h3ndrk h3ndrk force-pushed the horizontal-segments branch 4 times, most recently from 9137726 to c5f9335 Compare July 16, 2024 09:41
@rmburg rmburg marked this pull request as draft July 16, 2024 10:49
auto-merge was automatically disabled July 16, 2024 10:49

Pull request was converted to draft

@rmburg rmburg force-pushed the horizontal-segments branch from 765e4df to c5f9335 Compare July 16, 2024 11:44
@rmburg rmburg force-pushed the horizontal-segments branch 2 times, most recently from 77a1881 to 68ffeaa Compare July 16, 2024 16:44
@rmburg rmburg marked this pull request as ready for review July 16, 2024 16:44
@rmburg rmburg enabled auto-merge July 16, 2024 16:45
@rmburg rmburg removed the is:Dependent This is blocked because something else needs to happen first. label Jul 16, 2024
oleflb
oleflb previously requested changes Jul 17, 2024
@schluis schluis self-assigned this Jul 17, 2024
@rmburg rmburg force-pushed the horizontal-segments branch from 6c03d30 to 979eabc Compare July 17, 2024 07:15
@schluis schluis dismissed oleflb’s stale review July 17, 2024 08:24

I took over

@rmburg rmburg added this pull request to the merge queue Jul 17, 2024
Merged via the queue into HULKs:main with commit 5f2e83a Jul 17, 2024
25 checks passed
@rmburg rmburg deleted the horizontal-segments branch July 17, 2024 08:28
oleflb pushed a commit that referenced this pull request Sep 30, 2024
* Reimplement horizontal image segments

Co-authored-by: Hendrik <[email protected]>

* Filter horizontal segments

* Fix segmentation tests

* Fix review findings

* Fix zero length segments

---------

Co-authored-by: Hendrik <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants