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

Nice re-design, mobile problems. #6

Closed
radman-x opened this issue Mar 4, 2019 · 11 comments
Closed

Nice re-design, mobile problems. #6

radman-x opened this issue Mar 4, 2019 · 11 comments

Comments

@radman-x
Copy link

radman-x commented Mar 4, 2019

Nice re-design a few weeks ago, this is a big improvement of both libs. However, there seem to be some problems with running some of the demos (eg, asteriod) on a mobile device. We are getting missing textures, shader problems, etc. on some of the demos running on a Samsung S8. Have you tried the demos on a mobile phone?

@radman-x
Copy link
Author

radman-x commented Mar 4, 2019

One other point, the older version demos ran fine on the same mobile device.

@lo-th
Copy link
Owner

lo-th commented Mar 4, 2019

hum yes i try on S8 without problem's maybe your cache not refreshed ?
i have to improve mobile support i use new material ( lambert vs standard )
is pretty heavy for phone and also last dev version of three not complete.

@radman-x
Copy link
Author

radman-x commented Mar 4, 2019

Cleared cache and still not working (see asterioid attached). Strange that it runs on your S8...
1551717599664

@lo-th
Copy link
Owner

lo-th commented Mar 4, 2019

what mobile ? browser ?

@radman-x
Copy link
Author

radman-x commented Mar 4, 2019

Samsung Galaxy S8 (model SM-G955U1)
Android 8.0.0, Build/R16NW
Chrome 72.0.3626.105

@lo-th
Copy link
Owner

lo-th commented Mar 4, 2019

ha ok strange ? i have same version and model
( maybe is a american phone me is european not the same proc )
anyway i while update soon with some mobile optimisation
low background and environement.

@radman-x
Copy link
Author

radman-x commented Mar 4, 2019

OK, if I get a chance, will try to run remotely in debugger and see if any error shows up in the console. Also, in looking through the code a bit, I see that the material on the asteroid was changed from phong to standard, I may try changing it back and see what happens.

@lo-th
Copy link
Owner

lo-th commented Mar 4, 2019

yes maybe i change for phong on mobile, i have to try but standard is realy good
i make a update on lab (lower cube map reflection)

@radman-x
Copy link
Author

radman-x commented Mar 5, 2019

Another important piece of info: another member of our dev team ran the demos on his Samsung S7 and they seem to run fine! This seems to be something unique to my phone. It might be good to either wait for a few days until I can do the debugging above or, maybe, have some type of option in the AmmoLab configuration options like "basicMobile" that users can change. Like you said, it is nicer to have the standard material than the phong material and it appears that many phones can handle that.

@radman-x
Copy link
Author

After upgrading to Android 9, the current version of Ammo.lab now works fiine. So am closing this issue. Thanx for your prompt respo see.

@radman-x
Copy link
Author

radman-x commented Aug 1, 2021 via email

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

No branches or pull requests

2 participants