Skip to content

Comments

Remove slic package#45713

Merged
sloretz merged 1 commit intomasterfrom
sloretz__remove_slic
May 12, 2025
Merged

Remove slic package#45713
sloretz merged 1 commit intomasterfrom
sloretz__remove_slic

Conversation

@sloretz
Copy link
Contributor

@sloretz sloretz commented May 12, 2025

I noticed this package had no license when reviewing #45685 (review)

It seems like it comes from here https://github.com/PSMM/SLIC-Superpixels . Without a license it's not clear if the buildfarm can redistribute it in binary form. This PR removes the package from the buildfarm.

@k-okada FYI

https://github.com/jsk-ros-pkg/jsk_3rdparty/blob/a3b5eb682efc0f0e0ad6ca43faae4ad5a96867f3/3rdparty/slic/package.xml#L15

Signed-off-by: Shane Loretz <sloretz@intrinsic.ai>
@github-actions github-actions bot added the noetic Issue/PR is for the ROS 1 Noetic distribution label May 12, 2025
@sloretz sloretz merged commit 7467bfc into master May 12, 2025
5 checks passed
@sloretz sloretz deleted the sloretz__remove_slic branch May 12, 2025 16:51
@k-okada
Copy link
Contributor

k-okada commented May 13, 2025

thanks, I also removed <run_depend>slic</run_depned> from jsk_3rdparty/package.xml.
To note, there are unanswered question about SLIC license at PSMM/SLIC-Superpixels#14
SLIC algorithm is alredy implimented at opencv_contrib, so we could use this version PSMM/SLIC-Superpixels#5 (comment)

k-okada added a commit to tork-a/jsk_3rdparty-release that referenced this pull request May 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

noetic Issue/PR is for the ROS 1 Noetic distribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants