/incubator-mxnet

Fix incude error in plugin/opencv/cv_api.cc #9206

Merged
merged 1 commit into from Dec 27, 2017
+1 −1

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@mwbyeon @piiswrong
@mwbyeon
Contributor

mwbyeon commented Dec 27, 2017

Description

  • Fix include error in plugin/opencv/cv_api.cc (related #9203)
    • c_api_error.h does not exists. it replaced to c_api_common.h.

Checklist

Essentials

  • Passed code style checking (make lint)
  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage:
  • Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
  • Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
  • Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
  • Code is well-documented:
  • For user-facing API changes, API doc string has been updated.
  • For new C++ functions in header files, their functionalities and arguments are documented.
  • For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
  • To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

  • fix include path
@mwbyeon
fix incude error in plugin/opencv/cv_api.cc

All checks have passed

2 successful checks
@cjolivier01
continuous-integration/jenkins/pr-head — This commit looks good
Details
@cjolivier01
continuous-integration/jenkins/pr-merge — This commit looks good
Details
105768d

@mwbyeon mwbyeon referenced this pull request Dec 27, 2017

Open

compile error in plugin opencv #9203

@piiswrong piiswrong merged commit ad1ed7c into apache:master Dec 27, 2017
2 checks passed

2 checks passed

continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/jenkins/pr-merge This commit looks good
Details

meissnereric added a commit to meissnereric/incubator-mxnet that referenced this pull request Jan 2, 2018

@mwbyeon @meissnereric
fix incude error in plugin/opencv/cv_api.cc (#9206)
bfe2d84
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment