ReactVision / viro

ViroReact: The AR and VR library for React Native 📳💙💛🤍💚
MIT License
1.36k stars 157 forks source link

Expo app crashes / shuts down when opening a ViroARScene with Viro3DObject (Expo SDK 51) #298

Open jordyortega opened 3 months ago

jordyortega commented 3 months ago

Requirements:

Please go through this checklist before opening a new issue

Environment

Please provide the following information about your environment:

  1. Development OS: Mac
  2. Device OS & Version: iOS 16.7.8
  3. Version: ViroReact version 2.41.6 & React native 0.73.6 & Expo 51.0.1
  4. Device(s): All devices

Description

I recently updated to Expo SDK 51 and now when opening a ViroARScene with a Viro3DObject in it, it just crashes without an error message. When I remove the Viro3DObject, it just works. So it might have something to do with Viro3DObject :)

Reproducible Demo

`<ViroARScene onTrackingUpdated={(state, reason) => this.onInitialized(state, reason)}>

            <Viro3DObject
                source={coinobj}
                resources={[
                    coinmtl
                ]}
                type="OBJ"
                position={[0.0, -1.0, -3]}
                scale={[1, 1, 1]}
                hitSlop={{ top: 40, bottom: 40, left: 40, right: 40 }}
            />
        </ViroARScene>`
bastej commented 2 months ago

I noticed the same, looks like it about position props, when I got rid of it, app doesn't crashing

etiennemonnet commented 2 months ago

Hi, same for me, the position props is crashing the app

varun-raj commented 2 months ago

Hi, Same issue, getting rid of position stops crashing the app, anyone got a workaround?

gnvcor commented 1 month ago

I have same problem, prop "position" crashing the app (without errors and logs).

gnvcor commented 1 month ago

I tried all the available versions in npm (2.41.3, 2.41.4, 2.41.5, 2.41.6), every version has this error with position

gnvcor commented 1 month ago

It seems to be a system problem, guys please take a look!

rozekfr commented 1 month ago

Same here, I don't use Expo, but the latest RN 0.74.5. The position property crashes application.

gnvcor commented 1 month ago

@doranteseduardo please help)

gnvcor commented 1 month ago

Guys, i figured it out, the problem is in expo 51.

I downgraded expo to 50 and the position property works now.

how to downgrade expo: https://docs.expo.dev/workflow/upgrading-expo-sdk-walkthrough/#follow-the-release-notes-for-any-other-instructions

rozekfr commented 2 weeks ago

Any progres? I don't want to downgrade RN. Everything works on Android. It is just iOS problem.

@doranteseduardo @robertjcolley can you help please?

haakonjacobsen commented 1 week ago

Any updates/solutions to this with Expo 51?