Skip to content

Conversation

@mayavijx
Copy link

@mayavijx mayavijx commented Apr 4, 2022

  • Updated README documents.
  • Updated csharp sample with latest ImageSharp API
  • Updated cpp samples with nchw changes

@mayavijx mayavijx requested review from MaajidKhan and sfatimar April 4, 2022 13:47
Choose required opencv path. Skip the opencv flag if you don't want to build squeezenet sample.
**Note:**
If you are using the opencv from openvino package, below are the paths:
* For latest version (2022.1.0), run download_opencv.ps1 in \path\to\openvino\extars\script and the opencv will be downloaded at \path\to\openvino\extars.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pls correct the spellings.
1.download_opencv.sh
2.\extras (At 2 places)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected typo error at \extras. download_opencv.ps1 is correct as it is a powershell script in Windows.

```
**Note:**
If you are using the opencv from openvino package, below are the paths:
* For latest version (2022.1.0), run download_opencv.sh in /path/to/openvino/extars/script and the opencv will be downloaded at /path/to/openvino/extars.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

correct the spelling here
1./extras (At 2 places)

Choose required opencv path. Skip the opencv flag if you don't want to build squeezenet sample.
**Note:**
If you are using the opencv from openvino package, below are the paths:
* For latest version (2022.1.0), run download_opencv.ps1 in \path\to\openvino\extras\script and the opencv will be downloaded at \path\to\openvino\extras.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

download_opencv.ps1 should be download_opencv.sh

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants