Events by Week Performance Issues in v0.2.2 about the-blue-alliance-android HOT 5 CLOSED

Regarding events showing up in the wrong places, I'm going to call CNR - what did you have to do to get them to appear?

from the-blue-alliance-android.

Mainly spam the hell out of the week tabs while the year's events are still loading.

Here's a video of the 2007 glitch. https://www.youtube.com/watch?v=LzpoqDYlvh0

And another NPE that resulted. (Pastie: http://pastie.org/private/s98bkcuvdfvtd9roezvpq)
http://www.youtube.com/watch?v=2pu7p2rPqSY

EDIT: I've gotten Monkey to reproduce it, surprisingly lol. Then again, I'm not sure if a human could do 2000 random events this fast. https://www.youtube.com/watch?v=q98l1ZTTto

from the-blue-alliance-android.

I can fix the NPEs. I have no idea what to do about events appearing in the wrong place. It seems like calls are conflicting with each other and returning the next caller's data. But I think everything should be synchronized....

EDIT: That last NPE was also something already checked (it was moving too fast, so the activity must have gone out of scope between here and its null check). I don't think there's much more we can do on that front.

from the-blue-alliance-android.

The NPE may have something to do with the event coordinate locations being blank. I found a much simpler way to reproduce the crash that doesn't involve spamming random events. All of the 2011 events, except for the championship and offseason, don't have locations mapped to them, so they're more likely to get to the bugged line than the other years.

Process: Load up a non-championship/non-offseason event, but exit out before it finishes loading. NPE.

https://www.youtube.com/watch?v=62taqviv6eI
https://www.youtube.com/watch?v=uaMDu3ij5iM

EDIT: I think just simply adding a null check to activity.findViewById(R.id.event_location_container) in if(event.getLocation().isEmpty()) is probably the best way to go on that. NPE doesn't seem to appear after that.

from the-blue-alliance-android.

Alright, cool. Let's just make sure this gets thrown into the next bugfix PR.

from the-blue-alliance-android.