Skip to content

Sahar/csharp support openvino (refined)#4835

Merged
jywu-msft merged 5 commits intomasterfrom
openVinoChanges
Aug 18, 2020
Merged

Sahar/csharp support openvino (refined)#4835
jywu-msft merged 5 commits intomasterfrom
openVinoChanges

Conversation

@hariharans29
Copy link
Member

Description:
Refined version of #4703 which was reverted because it broke the Nuget pipeline. This change keeps the Nuget pipeline happy.

Contribution credit and fyi for the slight usage change of the is_linux_build flag: @sfatimar

CC: @jywu-msft

sfatimar and others added 2 commits August 17, 2020 16:35
* Temp changes and include openvino to ensure nuget package is created with linux till we configure azure ci pipeline

* string id change

* native nuget indentation changes

* documentation changes

* Update Openvino_execution_provider.md

Documentation includes openvino execution provider

* Update OpenVino-ExecutionProvider.md

update details to build csharp api for openvino execution provider .

* vadm backend revert

* Update Openvino-Execution-Provider.md

updated for review comments

* Update OpenVino-Execution-Provider.md

* Update OpenVINO-ExecutionProvider.md

* nuget package custome support for openvino
change in native nuget spec python script for including linux runtime

* change to make path to boolean flag

* removed the tab

* Update OpenVINO-ExecutionProvider.md

updated for review comments

* chnages to include pep8 warnings
modification to documentation

Co-authored-by: saharfraza <sfatima.3001@gmail.com>
Co-authored-by: sfatimar <sahar.fatima@intel/com>
@hariharans29 hariharans29 requested a review from a team as a code owner August 18, 2020 00:10
jywu-msft
jywu-msft previously approved these changes Aug 18, 2020
@jywu-msft
Copy link
Member

I guess this is failing PEP8 checks

@hariharans29
Copy link
Member Author

I guess this is failing PEP8 checks

Yes, fixed it. Hopefully, this run should pass.

@jywu-msft jywu-msft merged commit c878ecb into master Aug 18, 2020
@jywu-msft jywu-msft deleted the openVinoChanges branch August 18, 2020 04:52
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