Skip to content

Bug fix in OVEP csharp sample - #142

Merged
jywu-mysoft merged 17 commits into
microsoft:mainfrom
intel:shama/csharp_sample_update
Oct 27, 2022
Merged

Bug fix in OVEP csharp sample#142
jywu-mysoft merged 17 commits into
microsoft:mainfrom
intel:shama/csharp_sample_update

Conversation

@shamaksx

Copy link
Copy Markdown
Contributor

Updated OVEP csharp sample for a bug fix

@ghost

ghost commented Sep 20, 2022

Copy link
Copy Markdown

CLA assistant check
All CLA requirements met.

@shamaksx
shamaksx force-pushed the shama/csharp_sample_update branch from ee0c3c3 to 49e8ebd Compare September 28, 2022 10:26
/*
Copyright (C) 2021, Intel Corporation
SPDX-License-Identifier: Apache-2.0

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please add this line

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Line is added

@shamaksx
shamaksx force-pushed the shama/csharp_sample_update branch from 33cd5fb to ddd0e3e Compare September 28, 2022 10:36
nmaajidk and others added 14 commits September 28, 2022 03:43
…ckend (microsoft#120)

* Improve the sample with command line option to switch SNPE backend, and set the input file path.
Fix an issue for Android build, need to use libc++_shared.so from SNPE SDK.

* 1. Update the API call according the API change in Ort, SessionOptionsAppendExecutionProvider_SNPE -> SessionOptionsAppendExecutionProvider
2. format update
* update doc for snpe to reflect the API change
…frameworks is not added to the link list. Pending real fix from MAUI folks. (microsoft#131)

Also update ORT to 1.12.1 which has a better Android build.
In ResNet50DataReader, it uses an onnx session to obtain the model input shape. However it passes a madeup model name to the onnx session, resulting in file not found error.

This change provide the original float model path to the data reader
* sample notebooks for yolov4 and tiny-yoloV2

* folder restucturing for notebooks

* folder restucturing for notebooks

Co-authored-by: krishnendukx <krishnendux.kar@intel.com>
Co-authored-by: krishnendukx <111554749+krishnendukx@users.noreply.github.com>
Adding example run_qdq_debug.py
@shamaksx
shamaksx force-pushed the shama/csharp_sample_update branch from 4932a99 to 2747b0c Compare September 28, 2022 10:47
@sfatimar

Copy link
Copy Markdown
Contributor

@jywu-msft Please review

@shamaksx

Copy link
Copy Markdown
Contributor Author

@shamaksx the command you issued was incorrect. Please try again.

Examples are:

@microsoft-github-policy-service agree

and

@microsoft-github-policy-service agree company="your company"

@microsoft-github-policy-service agree company="intel"

@jywu-mysoft
jywu-mysoft merged commit 4999a7f into microsoft:main Oct 27, 2022
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.

10 participants