This repository was archived by the owner on Aug 22, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 630
C# Record and Playback APIs #822
Open
Brent-A
wants to merge
19
commits into
microsoft:develop
Choose a base branch
from
Brent-A:csharp-record
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
176e9e2
Added begining of record/playback for c#
Brent-A 3f2241e
Added record methods
Brent-A 6d1a588
Added minmial playback APIs
Brent-A 7209221
Completed Playback API
Brent-A 310287a
Added documentation
Brent-A ad6f6b6
Added some exception details
Brent-A c169698
Added tracing redirection to record
Brent-A f876d18
Enabled implicit stride in k4a_image_create
Brent-A 1797758
Removed default paramter in public API
Brent-A c6025bb
Added note on default parameters to standards.md
Brent-A 3768533
Merge branch 'issue-623' into csharp-record
Brent-A 972324a
Tests passing, and build config
Brent-A 594d672
Merge branch 'develop' of https://github.com/microsoft/Azure-Kinect-S…
Brent-A 0d50feb
Cleaned up C# style
Brent-A 09a6cf3
Merge branch 'develop' of https://github.com/microsoft/Azure-Kinect-S…
Brent-A 2b49c26
Style cleanup
Brent-A 05265bf
Added more options for native interop
Brent-A 5999e19
Added a few tests cases and fixed failures
Brent-A 3a3a059
Update src/csharp/Record/Exceptions/AzureKinectCreateRecordingExcepti…
Brent-A File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Added some exception details
- Loading branch information
commit ad6f6b67e7c39dd27fd7f48b7d6cd693b8e1dcd4
There are no files selected for viewing
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure the best way to deal with this. The "Any CPU" in the solution can get confusing, but Visual Studio really likes adding it back in.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure here either. Not supporting Any CPU in the nuget package is also pretty rough since downstream projects tend to have it by default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought we did have an issue tracking Any CPU support but I can't find it at this moment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.NET Core can actually do AnyCPU here. It's able to handle multiple native libraries with different architectures by deploying the native libs in the
\runtimes\win-x64\and\runtimes\win-x86\sub folders.