google-ar / arcore-unity-sdk

ARCore SDK for Unity
https://developers.google.com/ar
Other
1.4k stars 402 forks source link

Continuous error in logcat: ArLightEstimate_getEnvironmentalHdrAmbientSphericalHarmonics #614

Open BrahimHadriche opened 5 years ago

BrahimHadriche commented 5 years ago

SPECIFIC ISSUE ENCOUNTERED

Thrown every frame:

2019-07-31 13:55:28.011 24075-24103/<package name> E/Unity: MarshalDirectiveException: Cannot marshal type 'System.Single[0...,0...]'.
      at GoogleARCoreInternal.LightEstimateApi+ExternApi.ArLightEstimate_getEnvironmentalHdrAmbientSphericalHarmonics (System.IntPtr session, System.IntPtr light_estimate, System.Single[,] out_coefficients_27) [0x00000] in <00000000000000000000000000000000>:0 
      at GoogleARCoreInternal.LightEstimateApi+ExternApi.ArLightEstimate_getEnvironmentalHdrAmbientSphericalHarmonics (System.IntPtr session, System.IntPtr light_estimate, System.Single[,] out_coefficients_27) [0x00000] in <00000000000000000000000000000000>:0 
      at GoogleARCoreInternal.FrameApi.GetLightEstimate () [0x00000] in <00000000000000000000000000000000>:0 
      at GoogleARCore.Frame.get_LightEstimate () [0x00000] in <00000000000000000000000000000000>:0 
      at <Namespace.Class>.Update () [0x00000] in <00000000000000000000000000000000>:0 

VERSIONS USED

Packages: versionName=1.10.190603116 Hidden system packages: versionName=0

STEPS TO REPRODUCE THE ISSUE

1. 1. 1.

WORKAROUNDS (IF ANY)

ADDITIONAL COMMENTS

I guess the exception speaks for itself. https://developers.google.com/ar/reference/c/group/light#arlightestimate_getenvironmentalhdrambientsphericalharmonics

Marshalling of float *out_coefficients_27to c# float[,] out_coefficients_27 needs to be... fixed?

Edit: very important detail: This is with IL2CPP, Mono works fine.

BrahimHadriche commented 5 years ago

Since it works in Mono but not in IL2CPP, it can be considered as a bug in IL2CPP, but regardless, I think there should be a work around

alfiolocastro commented 5 years ago

Same here on a samsung S8, IL2CPP compile, Mono is confirmed working, but today everyone needs IL2CPP on android cause of arm64 support.

ggeutzzang commented 5 years ago

I also have same problem and error messages on arm64 / IL2CPP.. but, mono is ok.. I think this is a very serious problem. Since this month, Google Play only allows 64-bit apks to be uploaded. I hope it will be fixed soon.

alfiolocastro commented 5 years ago

I also have same problem and error messages on arm64 / IL2CPP.. but, mono is ok.. I think this is a very serious problem. Since this month, Google Play only allows 64-bit apks to be uploaded. I hope it will be fixed soon.

Did you encountered this issue in the latest 1.11?

andrelevi commented 5 years ago

Running into the same issue on IL2CPP builds with the latest v1.11.0 on a Pixel 1 and Samsung S8.

alfiolocastro commented 5 years ago

Any news? Considering that Google itself encourages/forces 64bit support to publish on PlayStore latest releases are unusable.