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

Seam carving is patented; remove from library #3646

Closed
stefanv opened this issue Jan 10, 2019 · 3 comments

Comments

Projects
None yet
3 participants
@stefanv
Copy link
Member

commented Jan 10, 2019

https://patents.google.com/patent/US8213745B2/en

Ref: #2808 #3082 #3643

This code may be made available outside of scikit-image under the same license, but our policy is not to include algorithms covered under active patents.

@stefanv stefanv added this to the 0.15 milestone Jan 10, 2019

@hmaarrfk

This comment has been minimized.

Copy link
Member

commented Jan 10, 2019

The cited paper is from 2007.

The priority date on the patent is from 2009.

I think you are safe @stefanv.

@hmaarrfk

This comment has been minimized.

Copy link
Member

commented Jan 10, 2019

Though it seems they cite a patent from the paper's authors, also from 2007. So :/

@jni

This comment has been minimized.

Copy link
Contributor

commented Jan 10, 2019

jni added a commit to jni/scikit-image that referenced this issue Feb 15, 2019

Remove seam carving
Seam carving is a patented algorithm and so cannot be distributed with
scikit-image, which must offer code that can be used "without charge and
without restriction".

Currently, attempting to run `seam_carve` will cause a
`NotImplementedError`, and then it can be removed altogether in 0.16.

Fixes scikit-image#3646

@jni jni referenced this issue Feb 15, 2019

Merged

Remove seam carving #3751

0 of 4 tasks complete

hmaarrfk added a commit that referenced this issue Feb 18, 2019

Remove seam carving (#3751)
* Remove seam carving

Seam carving is a patented algorithm and so cannot be distributed with
scikit-image, which must offer code that can be used "without charge and
without restriction".

Currently, attempting to run `seam_carve` will cause a
`NotImplementedError`, and then it can be removed altogether in 0.16.

Fixes #3646 

* Fix Python 2 support release note
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.