Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compatibility with Unity & Android #333

Open
wants to merge 28 commits into
base: master
Choose a base branch
from

Conversation

AuMilliat
Copy link

Proposed modifications:
Adding Microsoft.Psi.InteropAndroid, that is a reduction of Microsoft.Psi.Interop for TCPSource/Writer with a modification of dynamic to template (crash with refecerence exception on Quest).

Microsoft.Psi project modifications:
PsiStore path need to be changed for Android and check the permission to write at application level.
KnowSerializer instance writeable to allow the addition of ‘serializer’ and public to application level.
Mutex downgrade code & Removing null check operator to be compatible with Mono & Android.

Some modifications are unnecessary (or not complete) as the first target was to use Emitters and Receivers (commit ID: 446a0fa).

@AuMilliat
Copy link
Author

caa6e1b
Change the output of nuget packages into Builds/PsiPackages folder.

10df5c0 Merge differents modifications of Pipeline (ie CreateSynchedPipeline, switch some attributes from private to internal...) and add the RemotePipelineClockExporter/Importer components that allow to synch remote \psi applications from a dataset. (Our use case is to replay the datasets in our XR platforms).

@AuMilliat
Copy link
Author

AuMilliat commented Feb 5, 2025

Still 2 tests failing on my side:

  • SerializationTester.BoxedTest: on Assert.AreEqual(value, value.DeepClone());, SerializationTester.cs line 1000.
  • SerializationTester.DictionaryBackCompat : pass when run alone.

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.

1 participant