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

[WIP] add analyze method SLIC to generate superpixels from image #2

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

johnnychen94
Copy link
Owner

@johnnychen94 johnnychen94 commented Sep 29, 2019

TODO:

Reference:

[1] Radhakrishna Achanta, Appu Shaji, Kevin Smith, Aurelien Lucchi, Pascal Fua, and Sabine Süsstrunk, SLIC Superpixels, EPFL Technical Report no. 149300, June 2010.

[2] The original C++ codes

[3] scikit-image's slic

@codecov-io
Copy link

codecov-io commented Oct 10, 2019

Codecov Report

Merging #2 (26a7415) into master (3f85b16) will decrease coverage by 46.63%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master       #2       +/-   ##
===========================================
- Coverage   75.60%   28.97%   -46.64%     
===========================================
  Files           4        5        +1     
  Lines          41      107       +66     
===========================================
  Hits           31       31               
- Misses         10       76       +66     
Impacted Files Coverage Δ
src/SuperPixels.jl 100.00% <ø> (ø)
src/analyze.jl 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3f85b16...9ad8154. Read the comment docs.

@Cuda-Chen
Copy link

Hi @johnnychen94,
after few months I finally implement enforce connectivity features.
The implementation is based on scikit-image.

If you are pleased, I will send a PR

@johnnychen94
Copy link
Owner Author

This is so great a news to me! Please do so!

Add enforce connectivity feature in analyze method.
The enforce connectivity implementation is based on [scikit-image](https://github.com/scikit-image/scikit-image/blob/7e4840bd9439d1dfb6beaf549998452c99f97fdd/skimage/segmentation/_slic.pyx#L240-L348).

Because the original implementation uses 0-indexing, so I tried to translate into 1-indexing.
Though so far there are no any problems about `unbounded error`, it is ought to conduct test because I think
the implementation still have some problems in BFS part.
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.

3 participants