Skip to content

Content quiz performance#97

Open
nononovia wants to merge 17 commits into
mainfrom
Content_quiz_performance
Open

Content quiz performance#97
nononovia wants to merge 17 commits into
mainfrom
Content_quiz_performance

Conversation

@nononovia

Copy link
Copy Markdown
Collaborator

Refactored code for processing number of content quiz attempts to utilize util functions.
Added calculation and graphing for content quiz first attempt grade.
Generates both by page and by module plots for both extra attempts and first attempt grade.

Comment thread scripts/content_quiz_analysis/analysis_on_content_quiz.py Outdated
Comment thread util/__init__.py
Comment thread util/reading_logs.py Outdated
Comment thread util/reading_logs.py Outdated
Comment thread util/reading_logs.py Outdated
Comment thread util/reading_logs.py Outdated
@Opeyem1a Opeyem1a assigned Opeyem1a and nononovia and unassigned Opeyem1a Feb 7, 2022
…culations. Eg. overall on module graph is calculated by averaging the mean of each module and calculating the standard deviation of the mean of each module, rather than getting each student's submission and averaging it across all page and modules.
@nononovia
nononovia requested a review from Opeyem1a February 7, 2022 11:19
Comment thread schemas/course.py
Comment thread util/reading_logs.py
cols = [col for col in page_content_quiz_df if col.startswith('q')]
questions_df = page_content_quiz_df[cols]

def first_attempt_grade(row):

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You could probably generalize this inner function since it's being used twice I think

Comment thread util/reading_logs.py
Retrieves the average number of content quiz attempts before a correct answer for a page. If given a
data448_id, only retrieves that student's reading speed for the page. Without a data448_id, retrieves the
average reading speed of that page.
average number of attempts of that page.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch

Comment thread util/__init__.py
from .canvas_api import setup_submissions_filepath, get_quiz_id_from_file_name, DateTimeEncoder
from .data_cleaner import keep_latest_survey_attempt
from .util import normalize
from .plots import set_plot_settings

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's dangerous to auto-format __init__.py files since it can create circular imports

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