-
Notifications
You must be signed in to change notification settings - Fork 4
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
Usage counts are actually UsageRevision counts #130
Comments
Still an issue: At http://flat.nert.georgetown.edu/ko/, it says hanthey has 14 construals But http://flat.nert.georgetown.edu/ko/%ED%95%9C%ED%85%8C/ shows 4 construals |
i could take a look at this? |
You could try. It involves a complicated query. |
Looking at the generated SQL, i thought of a fix where the distinct construal list in the usage revision table is used for the counts. The hanthey count in both the pages is now 4, and the German 'an' count is 7. I've attached the SQL generated by django, after the fix. Raised a PR #217 |
The counts are still not always correct: e.g. http://flat.nert.georgetown.edu/ko/ shows while http://flat.nert.georgetown.edu/ko/%EC%97%90/ lists only 9 usages. (Could the 10th be a deprecated/deleted one?) |
See b17c7bb#commitcomment-29874556
1126fd4 may solve the problem but the solution requires Django 2.x. (#45)
The text was updated successfully, but these errors were encountered: