Closed kjdav closed 11 years ago
kevinburleigh75
commented
11 years ago Due to the lack of feedback timing snafu, this is only possible for student exercises for which feedback viewing is required for credit. Does that seem adequate for now?
jpslav
commented
11 years ago Seems reasonable to me.
On Feb 11, 2013, at 5:18 PM, kevinburleigh75 notifications@github.com wrote:
Due to the lack of feedback timing snafu, this is only possible for student exercises for which feedback viewing is required for credit. Does that seem adequate for now?
— Reply to this email directly or view it on GitHub.
kjdav
commented
11 years ago I think so, too.
Students who have to view within a particular window will see a different message than those who can view for credit anytime, right? Something like Viewed for Credit vs just Viewed, etc
On Mon, Feb 11, 2013 at 9:22 PM, JP Slavinsky notifications@github.comwrote:
Seems reasonable to me.
On Feb 11, 2013, at 5:18 PM, kevinburleigh75 notifications@github.com wrote:
Due to the lack of feedback timing snafu, this is only possible for student exercises for which feedback viewing is required for credit. Does that seem adequate for now?
— Reply to this email directly or view it on GitHub.
— Reply to this email directly or view it on GitHubhttps://github.com/lml/ost/issues/202#issuecomment-13415190.
kevinburleigh75
commented
11 years ago Does this seem reasonable for now? Or should we try to make little circles like the Educator view (in which case we'd need a legend, etc.)?
kjdav
commented
11 years ago Cool. Feedback credit not given is the only text that gives me pause -- students will see that and wonder why no credit. I might say something like, feedback viewed vs feedback not viewed for classes where you get credit for viewing it anytime. And for classes with a credit window, show them those messages for credit/no credit, plus "feedback viewed but late." Thoughts?
For 3.0 -- imo icons would be nicer; this is a lot of text. I think this will be helpful for now but have ideas for better ways to display all the info we want students to be able to see for each exercise in 3.0.
kevinburleigh75
commented
11 years ago I thought it was more important to emphasis when the students did/didn't receive credit for viewing feedback. I also didn't want to confuse "feedback credit" with "correctness credit".
kevinburleigh75
commented
11 years ago After some back-and-forth with Kim, we decided that the following wording is slightly better since it emphasizes the desired behavior (view feedback) over the reward (get credit):
jpslav
commented
11 years ago If feedback not required what is the message when viewed?
On Feb 12, 2013, at 11:24 AM, kevinburleigh75 notifications@github.com wrote:
After some back-and-forth with Kim, we decided that the following wording is slightly better since it emphasizes the desired behavior (view feedback) over the reward (get credit):
— Reply to this email directly or view it on GitHub.
kevinburleigh75
commented
11 years ago There is no message, since it cannot be reliably determined if the student viewed feedback under those circumstances. This is due to the combination of feedback_credit_multiplier being 1.0 by default and the lack of feedback ResponseTimes for some assignments. That's why this feature only applies to assignments where viewing feedback is required.
jpslav
commented
11 years ago perfect
On Feb 12, 2013, at 12:50 PM, kevinburleigh75 notifications@github.com wrote:
There is no message, since it cannot be reliably determined if the student viewed feedback under those circumstances. This is due to the combination of feedback_credit_multiplier being 1.0 by default and the lack of feedback ResponseTimes for some assignments. That's why this feature only applies to assignments where viewing feedback is required.
— Reply to this email directly or view it on GitHub.
jpslav
commented
11 years ago Thanks, @kevinburleigh75, merged and deploying
On Feb 12, 2013, at 12:50 PM, kevinburleigh75 notifications@github.com wrote:
There is no message, since it cannot be reliably determined if the student viewed feedback under those circumstances. This is due to the combination of feedback_credit_multiplier being 1.0 by default and the lack of feedback ResponseTimes for some assignments. That's why this feature only applies to assignments where viewing feedback is required.
— Reply to this email directly or view it on GitHub.
Could add some text like Viewed or Not Yet Viewed. Or maybe an icon, like a red flag to signify To Do or green check to signify viewed. I like icons, though maybe better to plan it out better for OST 3.0.
For classes where there is a window to view feedback for credit, would also be useful for students to see if they viewed feedback during the window or not.