-
Notifications
You must be signed in to change notification settings - Fork 6.8k
[v1.9.x] Remove cpp-predict example due to licensing issues #20530
[v1.9.x] Remove cpp-predict example due to licensing issues #20530
Conversation
Hey @josephevans , Thanks for submitting the PR
CI supported jobs: [clang, unix-gpu, website, miscellaneous, sanity, centos-cpu, centos-gpu, windows-cpu, edge, windows-gpu, unix-cpu] Note: |
@leezu @sandeep-krishnamurthy @szha Could you please review? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we just delete the file from the release archive?
Good point, makes it much easier. Refactored PR.
fab402f
to
05a9c18
Compare
This fixes item 3 in #20475. |
Just remove image classification CPP example from source tarball. (apache#20530)
Just remove image classification CPP example from source tarball. (#20530)
Just remove image classification CPP example from source tarball. (apache#20530)
This fixes #20515 - remove cpp example for image classification that is not compliant with Apache 2.0 license due to prior copyright.
This change removes the example from the release source archive.