Skip to content

Fix demo scripts#6280

Merged
ilya-lavrenov merged 4 commits intoopenvinotoolkit:masterfrom
ilya-naumov:fix-demo-scripts
Jun 21, 2021
Merged

Fix demo scripts#6280
ilya-lavrenov merged 4 commits intoopenvinotoolkit:masterfrom
ilya-naumov:fix-demo-scripts

Conversation

@ilya-naumov
Copy link
Copy Markdown
Contributor

Details:

Fix some problems related to #6141

  • Errors like ERROR: unknown command line flag 'no_show ' and ERROR: unknown command line flag 'niter1000'
  • Raw escape characters output:
    \n\n###################################################\n\nRun Inference Engine security_barrier_camera demo

@openvino-pushbot openvino-pushbot added the category: CI OpenVINO public CI label Jun 21, 2021
@ilya-lavrenov ilya-lavrenov requested a review from IRDonch June 21, 2021 12:35
@ilya-lavrenov ilya-lavrenov added this to the 2022.1 milestone Jun 21, 2021
Copy link
Copy Markdown

@IRDonch IRDonch left a comment

Choose a reason for hiding this comment

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

LGTM (although I personally would've changed dashes and run_again to contain real newlines (via $'') instead of switching to echo -ne).

@ilya-lavrenov ilya-lavrenov enabled auto-merge (squash) June 21, 2021 16:41
@ilya-lavrenov ilya-lavrenov merged commit 4dc9c03 into openvinotoolkit:master Jun 21, 2021
@ilya-lavrenov
Copy link
Copy Markdown
Contributor

@ilya-naumov thanks!

rnugmanx pushed a commit to rnugmanx/openvino that referenced this pull request Aug 26, 2021
* Fix unknown command line flag errors if -sample-options is used

* Fix raw escape sequences output with dashes

* Fix raw escape sequences output with run_again message

* try change sampleoptions to array
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: CI OpenVINO public CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants