Skip to content

add rtlucassen slide segmenter to trident backend - #58

Open
siemdejong wants to merge 1 commit into
mainfrom
slidesegmenter-trident
Open

siemdejong wants to merge 1 commit into
mainfrom
slidesegmenter-trident

Conversation

@siemdejong

Copy link
Copy Markdown
Contributor

fixes #57

@siemdejong

Copy link
Copy Markdown
Contributor Author
image

@siemdejong

siemdejong commented Sep 7, 2026

Copy link
Copy Markdown
Contributor Author

It's not super clean, especially because it is copied to gpu and to cpu and gpu and cpu again. Moreover, it needs to stitch the patches together, but I don't see that negatively impacting the resulting geojson on first glance. Also, it needs a for loop in forward(), which is undesirable. Future work could write a similar method to SlideSegmenter.segment() that takes in a stack of patches.

This commit does provide a solution to the bottleneck.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SlideSegmenter goes out of memory on larger slides

1 participant