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

"Use prepared questions" screen can take time to load #109

Open
chrqls opened this issue Jan 6, 2014 · 4 comments
Open

"Use prepared questions" screen can take time to load #109

chrqls opened this issue Jan 6, 2014 · 4 comments

Comments

@chrqls
Copy link

chrqls commented Jan 6, 2014

The problem

If we have a lot of IQSets preloaded in IQManager with sometimes images in them (in my case 9 IQSets containing 7-8 questions + an IQSet of 23 questions), the screen "Use prepared questions" can take a lot of time to load (between 20-30s to display).

The reason

If we have 10 iqsets, the teacher app makes 10 requests /smile/iqset/<key> to the server to get specific values such as the number of questions or the data to preview the iqset ( because /smile/iqsets does not deliver these values)

The solution

To fix this performance issue, that might be interesting to prepare these data directly from the server, at least adding the size of each iqset in /smile/iqsets

@truedat101
Copy link
Contributor

Can you quantify how long the wait time is?

@truedat101
Copy link
Contributor

If wait is longer than 3 seconds, we should put loading into an AsyncTask, and put up a "progressbar" type dialog.

@chrqls
Copy link
Author

chrqls commented Jan 22, 2014

I have to find a way to get the size of an iqset directly from the server in a JSON request instead of calculate the size of each IQSet on teacher tablet.
In my opinion, this is not a job that should be done on android side.
I have to see if I can change on node_smile_teacher the JSON returned when I load the list of iqset.

@truedat101
Copy link
Contributor

Depends on how your get the iqset. One way is to get it as a JSON object, and another way is to get it as a list.

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