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

Reference test verse selection #293

Open
alexcwatt opened this issue Aug 24, 2014 · 3 comments
Open

Reference test verse selection #293

alexcwatt opened this issue Aug 24, 2014 · 3 comments

Comments

@alexcwatt
Copy link
Collaborator

Instead of random sorting in MemversesController#test_next_ref, we should probably sort by last tested.

But in these 20 or more questions, there were about 5 verse references that were questioned more than 3 times each, while it could have drawn on other verses instead.

source.

@Nateowami
Copy link
Collaborator

Last tested as in "verse last tested" or "reference last tested"? As far as I can tell the latter is not being tracked, but it could be added. The other thing is that some users could have reviewed every verse quite recently, and if only the day of review were tracked, it could be hard to choose verses for users who only have a few. My thought would be to factor in both last test and interval.

@alexcwatt
Copy link
Collaborator Author

We can have an SQL query get next test minus current interval for
references, which are tracked.
On Aug 25, 2014 8:54 AM, "Nateowami" [email protected] wrote:

Last tested as in "verse last tested" or "reference last tested"? As far
as I can tell the latter is not being tracked, but it could be added. The
other thing is that some users could have reviewed every verse quite
recently, and if only the day of review were tracked, it could be hard to
choose verses for users who only have a few. My thought would be to factor
in both last test and interval.


Reply to this email directly or view it on GitHub
#293 (comment).

@Nateowami
Copy link
Collaborator

That totally makes sense; I don't know why I didn't think of that.

The accuracy test has the same issue (though not as badly I think), but I doubt that'd be as easy to fix (it's currently completely random).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants