Update Xamarin sample code#9925
Merged
Merged
Conversation
Remove fasterrcnn xamarin sample. Adding a more complete xamarin sample targeted at end users to the inference examples repo. See microsoft/onnxruntime-inference-examples#58
edgchen1
reviewed
Dec 6, 2021
| - build it yourself | ||
| - [Android](https://onnxruntime.ai/docs/build/android.html) build instructions | ||
| - [iOS](https://onnxruntime.ai/docs/build/ios.html) build instructions | ||
| - extract if from the Microsoft.ML.OnnxRuntime nuget package using [NuGetPackageExplorer](https://github.com/NuGetPackageExplorer/NuGetPackageExplorer) |
Contributor
There was a problem hiding this comment.
Suggested change
| - extract if from the Microsoft.ML.OnnxRuntime nuget package using [NuGetPackageExplorer](https://github.com/NuGetPackageExplorer/NuGetPackageExplorer) | |
| - extract it from the Microsoft.ML.OnnxRuntime nuget package using [NuGetPackageExplorer](https://github.com/NuGetPackageExplorer/NuGetPackageExplorer) |
edgchen1
reviewed
Dec 6, 2021
| - extract if from the Microsoft.ML.OnnxRuntime nuget package using [NuGetPackageExplorer](https://github.com/NuGetPackageExplorer/NuGetPackageExplorer) | ||
| - release version is [here](https://www.nuget.org/packages/Microsoft.ML.OnnxRuntime/) | ||
| - integration test version is [here](https://int.nugettest.org/packages/Microsoft.ML.OnnxRuntime/) | ||
| - this is frequently update and should work if you're currently using the `master` branch of ONNX Runtime |
Contributor
There was a problem hiding this comment.
Suggested change
| - this is frequently update and should work if you're currently using the `master` branch of ONNX Runtime | |
| - this is frequently updated and should work if you're currently using the `master` branch of ONNX Runtime |
edgchen1
previously approved these changes
Dec 6, 2021
guoyu-wang
approved these changes
Dec 7, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description:
Update InferenceSample Xamarin components so that mobile app is more user friendly. Previously it just wrote to Console and had a confusing static UI that didn't make it clear what was happening. Now it has a button to run the tests, and progress updates.
Remove fasterrcnn xamarin sample. Adding a more complete xamarin sample targeted at end users to the inference examples repo.
Motivation and Context
Cleanup the Xamarin sample setup.