markoshorro / look-around Goto Github PK

Cuando estas en la aplicacion y pasas a la pestaña VR y luego vuelves al mapa, no te guarda la posicion que tenias.

He comprobado que el proceso de lo que es la captura de pantalla es correcto, pero algo debo de estar haciendo mal que me saca la captura solo del botón y no de la pantalla completa.

Habíamos comentado lo de meterle una imagen al ListView del local donde sea el evento, ¿no?

Esto lo habíamos hablado en su momento, no sé si es relevante ahora mismo.

• Antes la petición de permisos la hacía Vuforia. Como no nos gustaba que lo hiciera Vuforia, ahora las hacemos nosotros, pero sólo la primera vez que se inicia, ¿está bien? ¿Habría que pedirlas a medida que usas la app en caso de no disponer de los permisos? Mi pregunta es, ¿es buena nuestra estrategia?

Están pasando varias cosas:

1. Los eventos se están pidiendo con demasiada frecuencia (timer?)
2. Hay duplicados en la lista de eventos encontrados (algún tipo de chequeo antes de añadir un nuevo evento?)
3. Vamos a incluír detalles?
4. Vamos a incluir marcadores en el mapa?

You will notice that the app is working very slow. This started to happen when the location feature was installed. I guess this should work as a service, rather than in main thread.

Need to fix it!

gal.udc.evilcorp.lookaround E/AndroidRuntime: FATAL EXCEPTION: main
                                                                             Process: gal.udc.evilcorp.lookaround, PID: 26533
                                                                             java.lang.Error: FATAL EXCEPTION [main]
                                                                             Unity version     : 5.5.2f1
                                                                             Device model      : Sony F3111
                                                                             Device fingerprint: Sony/F3111/F3111:6.0/33.2.A.3.81/2432859652:user/release-keys
                                                                             
                                                                             Caused by: java.lang.OutOfMemoryError: pthread_create (1040KB stack) failed: Try again
                                                                                 at java.lang.Thread.nativeCreate(Native Method)
                                                                                 at java.lang.Thread.start(Thread.java:1063)
                                                                                 at com.android.volley.RequestQueue.start(RequestQueue.java:152)
                                                                                 at com.android.volley.toolbox.Volley.newRequestQueue(Volley.java:66)
                                                                                 at gal.udc.evilcorp.lookaround.util.GeolocationService.requestEvents(GeolocationService.java:456)
                                                                                 at gal.udc.evilcorp.lookaround.util.GeolocationService.getEventsByPlaces(GeolocationService.java:396)
                                                                                 at gal.udc.evilcorp.lookaround.util.GeolocationService.parsePlaces(GeolocationService.java:380)
                                                                                 at gal.udc.evilcorp.lookaround.util.GeolocationService.access$400(GeolocationService.java:72)
                                                                                 at gal.udc.evilcorp.lookaround.util.GeolocationService$3.onResponse(GeolocationService.java:344)
                                                                                 at gal.udc.evilcorp.lookaround.util.GeolocationService$3.onResponse(GeolocationService.java:338)
                                                                                 at com.android.volley.toolbox.JsonRequest.deliverResponse(JsonRequest.java:65)
                                                                                 at com.android.volley.ExecutorDelivery$ResponseDeliveryRunnable.run(ExecutorDelivery.java:99)
                                                                                 at android.os.Handler.handleCallback(Handler.java:815)
                                                                                 at android.os.Handler.dispatchMessage(Handler.java:104)
                                                                                 at android.os.Looper.loop(Looper.java:207)
                                                                                 at android.app.ActivityThread.main(ActivityThread.java:5763)
                                                                                 at java.lang.reflect.Method.invoke(Native Method)
                                                                                 at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:888)
                                                                                 at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:749)

Hay problemas de memoria. Habríua que controlar el número de peticiones que se realizan porque por lo que veo en el log se hacen demasiadas, y si empezamos a lanzar tantos threads (lo que hace Volley) habría que controlar que no exceda la memoria.
Como soluciones se me ocurre:

1. Meter el servicio como proceso aparte
2. Algún mecanismo para comprobar el estado de la memoria (no sé cuán factible es)
3. Controlar que no se hagan peticiones "sin rumbo"

It is needed to redo the code and use android.hardware.camera2 instead for obvious reasons.

Ya he metido la parte de Vuforia en la aplicación de Android existente, pero da algunos problemas la integración. He conseguido que no den errores y se pueda compilar pero cuando se ejecuta la aplicación se generan dos iconos de la app en el móvil. Por lo que he estado viendo, esto es debido a que el Android Manifest tiene los tags que deberían de cambiarse por otra cosa y eso, en principio, resolvería el problema.

¿Realmente vamos a necesitar implementar algo relacionado con la cámara? Lo estoy pensando porque Vuforia ya tiene "ese código" implementado implícitamente.

Esta nota la vi en el Roadmap que tenemos y no estaba seguro de si sería buena idea mantenerla o no.

Bueno, creo que hemos experimentado todos este comportamiento después de usar el fragment de vuforia. Creo que el problema es que Vuforia no "libera" bien la vista y cuando accedes a las otras vistas, lo cual no me tiene demasiado sentido y me está llevando por el camino de la amargura...

A ver si consigo algún workaround pero de momento no sé...

Estoy desarrollando una mejora para hacer que exista una primera pantalla la primera vez que se inicia la aplicación para mostrar un mensaje de bienvenida y de las features de la aplicación (como hacen muchas otras para explicar las características más destacadas).

Además, podríamos pensar en meter aquí la petición de permisos de manera que se hiciera de manera ordenada.

El problema de los APIs no sé en qué punto meterlo... ¿hemos hecho pruebas en Android 4.0? En caso de que funcione en Android 4.0, cerraría esto.

We should update the interface, remove the ugly button at the bottom of the screen and try to organize and adapt the elements shown in the screen to fit any size.

Básicamente lo que hice fué meter el textview en el activity_main.xml para que no diese el fallo que nos daba antes con el parent. Ahora falla porque cuando hago "final TextView textView = (TextView) findViewById(R.id.location);" dice que es null. Buscando encontré lo siguiente:

The only reason for findViewById to return null if you are passing a valid id is that you are either setting the wrong content view (with setContentView) or not setting a content view at all.