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

LineSegmentDetector needs to be removed for license #961

Open
sciencewhiz opened this issue Feb 5, 2020 · 6 comments
Open

LineSegmentDetector needs to be removed for license #961

sciencewhiz opened this issue Feb 5, 2020 · 6 comments

Comments

@sciencewhiz
Copy link

Per https://www.chiefdelphi.com/t/cpp-multicameraserver-wont-build/375965/7, GRIP generates code that uses LineSegmentDetector. Per opencv/opencv_contrib#2016, LineSegmentDetector used Affero GPL v3 code and was removed. GRIP needs to remove the calls to LineSegmentDetector or relicense to a compatible license.

@virtuald
Copy link

virtuald commented Feb 5, 2020

If GRIP generates the code but doesn't use it, doesn't that mean the end user's code is AGPLv3, not GRIP?

... but still, removing it would be a good idea.

@sciencewhiz
Copy link
Author

It's also used in GRIP to show the results.

@JLLeitschuh
Copy link
Member

If it's incompatible with BSD, we need to remove our usage too.
CC: @SamCarlberg @AustinShalit

@JLLeitschuh
Copy link
Member

More information here:
opencv/opencv_contrib#2524

@ThadHouse
Copy link
Contributor

opencv/opencv_contrib#2016

Based on the latest post, it seems like the author realized the license issue was a mistake, and relicensed the code. So anything shipping with 3.x should be safe.

@GrantPerkins
Copy link
Contributor

#976 migrates to FastLineDetector from LineSegmentDetector

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

No branches or pull requests

5 participants