Skip to content
This repository has been archived by the owner on May 10, 2022. It is now read-only.

Let feedback viewing penalty be configurable #346

Open
jpslav opened this issue May 20, 2014 · 0 comments
Open

Let feedback viewing penalty be configurable #346

jpslav opened this issue May 20, 2014 · 0 comments
Assignees

Comments

@jpslav
Copy link
Member

jpslav commented May 20, 2014

In certain research conditions students are required to view feedback in order to receive credit for having worked an exercise. When this option is set now, if a student does not view required feedback in the allotted window, they don't get any credit for the exercise. Some instructors have requested that the penalty for not viewing feedback be something less than losing 100% of the credit for an exercise.

This task is to make the penalty for not viewing feedback a configurable fraction between 0 and 1. A penalty of 1 results in a loss of 100% of the earned credit for an exercise. A penalty of .25 results in a loss of 25% of the earned credit for an exercise.

This will require changes to the FeedbackCondition model, views in app/views/feedback_conditions, and likely changes in the StudentExercise model (where the overall credit is calculated). Note also that there is a "score detail" view somewhere that should be updated to reflect the amount of the penalty (previously assumed to always be 100%).

I imagine that the views should show the penalty as a percent. The default value should be 100%, or you can not have a default and require that the user choose a value when requiring feedback viewing. Also, @kjdav should write a help topic for where the percent is entered (search for link_to_help to give @kjdav a placeholder in the code)

The developer should consider if the feedback penalty should be recorded in the research report (doesn't always apply so may not be great to do this; researchers can always get the information out by encoding the feedback percentage in the learning condition label).

cc @kjdav @Dantemss

navilan added a commit to navilan/ost that referenced this issue Jun 12, 2014
- Add feedback penalty field to the settings store.
- Add feedback penalty to show and edit views.
- Add JS to show / hide penalty based on feedback required.
- Add help file for @kjdav with some help text.
- Ensure score detail page does not need any updates.
navilan added a commit to navilan/ost that referenced this issue Jun 12, 2014
- Add feedback penalty field to the settings store.
- Add feedback penalty to show and edit views.
- Add JS to show / hide penalty based on feedback required.
- Add help file for @kjdav with some help text.
- Ensure score detail page does not need any updates.
navilan added a commit to navilan/ost that referenced this issue Jun 12, 2014
- Add feedback penalty field to the settings store.
- Add feedback penalty to show and edit views.
- Add JS to show / hide penalty based on feedback required.
- Add help file for @kjdav with some help text.
- Ensure score detail page does not need any updates.
navilan added a commit to navilan/ost that referenced this issue Jun 13, 2014
- Stop getting confused between JS and CS.
- Fix copyrights on changed files.
navilan added a commit to navilan/ost that referenced this issue Jun 28, 2014
- Use `unless` instead of `if not`
- Use floating point division.
jpslav added a commit that referenced this issue Jun 29, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants