dcv icon indicating copy to clipboard operation
dcv copied to clipboard

Implementation of the skeletonize function for binary regions

Open aferust opened this issue 2 years ago • 5 comments

This is a rewrite of scikit-image's 2D fast skeletonized algorithm[1] written in cython. An example usage is also included in the examples folder.

1: https://github.com/scikit-image/scikit-image/blob/main/skimage/morphology/_skeletonize_cy.pyx

aferust avatar Oct 19 '22 08:10 aferust

We need a reviewer for DCV

9il avatar Oct 19 '22 10:10 9il

Sure. There must be an invite for review feature that I did not use before. Maybe you can invite the last contributors or PR owners for this purpose. Sadly, it looks like there are not many people involved in CV stuff within the D community.

aferust avatar Oct 19 '22 10:10 aferust

@ljubobratovicrelja Do you wish to review the code? Otherwise, could @aferust continue maintain the repo in mir?

9il avatar Oct 19 '22 12:10 9il

Hi @aferust and @9il.

I am very happy to allow Ferhat to take over the maintenance of this project, as in number of occasions I've tried and failed to return to this project.

I'd gladly give this a review, but in order to do that properly, I'd have to clone the code and run in on my end, as I was for so long away from this project. I tried on Ferhat's last PR, and I didn't manage.

Therefore, Ferhat if you are willing to take over, please let me know and I'll happily give you the rights.

Cheers, Relja

ljubobratovicrelja avatar Oct 19 '22 18:10 ljubobratovicrelja

Dear Relja and Ilia, I can accept the ownership since DCV does not have too much traffic (PRs, issue reports, etc.), and it will not have it shortly I guess. I can carry the flag for a while. Please also note that this does not mean I can solve all the pre-existing issues related to the library shortly. I would even transfer it to someone else who is more willing to do it someday.

aferust avatar Oct 20 '22 10:10 aferust

@aferust sorry for not responding earlier, I am on a business trip with very little time for personal email, so your message skipped me. I've given you 'Maintainer' rights, hopefully this means that you can merge PRs by yourself. Please let me know if that's not the case.

ljubobratovicrelja avatar Oct 31 '22 15:10 ljubobratovicrelja