Skip to content

Conversation

@ksteph
Copy link
Contributor

@ksteph ksteph commented Aug 23, 2013

Makes new tables that are queryable to get aggregate data on student per
problem information and student grades at three levels: overall course,
assignment type, and assignment.

Finally got the queryable app in shape for a good review.
Test coverage: 99%

@jrbl @jbau @dcadams

Makes new tables that are queryable to get aggregate data on student per
problem information and student grades at three levels: overall course,
assignment type, and assignment.
@dcadams dcadams closed this Sep 6, 2013
chrisrossi pushed a commit to jazkarta/edx-platform that referenced this pull request Mar 31, 2014
…ueue

Make it so we don't require HTTP basic auth to talk to the queue server -- I'm merging my own PR, so please check me -- I just need to do this to get the sandbox testing happy for 6.00.
@sefk sefk deleted the ksteph/edx-west/release/queryable branch June 23, 2014 04:48
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request May 9, 2016
Fix requirements for gacco-jwplayerxblock
mtyaka referenced this pull request in open-craft/openedx-platform Jun 13, 2017
caesar2164 added a commit to caesar2164/edx-platform that referenced this pull request May 18, 2018
…ng-profile-name

Encodes staff profile names as they can be unicode
andrey-canon pushed a commit to eduNEXT/edx-platform that referenced this pull request Jun 25, 2018
Sujeet1379 pushed a commit to chandrudev/edx-platform that referenced this pull request Nov 17, 2022
It was accidentally removed when we switched to the external
header (frontend-component-header), but the string is still
actually used.
johanseto pushed a commit to nelc/edx-platform that referenced this pull request Jan 22, 2024
DanielVZ96 referenced this pull request in open-craft/openedx-platform Jan 31, 2024
- `duration` changes name to `duration_ms` (more explicit)
- duration is now correct, as opposed to being milliseconds modulo seconds

The old usage metrics calculation for command duration used the
microseconds property of a timedelta, which only gets the microseconds
*component* of the difference -- not the difference *in* microseconds.
(It was then divided by 1000 to get milliseconds.)

The new calculation takes the difference and then divides it by
milliseconds. (This continues to compute the floor, using integer
division.)

Since the previous data was unusable, the name change doesn't interfere
with any analysis we were doing.
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

Successfully merging this pull request may close these issues.

3 participants