Skip to content

Clarify how timing rules work in facilities. #6

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.

Sign up for GitHub

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

Open
davis68 wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Clarify how timing rules work in facilities. #6

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
3048790
Clarify how timing rules work in facilities.
davis68 Jan 31, 2017
dfc4f13
Merge pull request #1 from inducer/master
davis68 Dec 15, 2018
3754df8
Create f.m
davis68 Dec 15, 2018
32330c6
Post Octave/Python flow.
davis68 Apr 9, 2019
7e3a8a9
Post Octave/Python flow.
davis68 Apr 9, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 15 additions & 11 deletions flows/exam-1.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,24 +28,28 @@ rules:
permissions: []

-
if_in_facility: test_center
if_session_duration_shorter_than_minutes: 0.5
permissions: [view, submit_answer, end_session, see_session_time, lock_down_as_exam_session]
if_has_role: [instructor]
permissions: [view, submit_answer, change_answer, end_session, see_correctness, see_session_time]

@inducer inducer Feb 13, 2019

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

change_answer and see_correctness is not a good combo if using multiple-choice. I think it's preferable to add change_answer using per-page permissions.


-
if_in_facility: test_center
message: "You exam will end soon."
if_session_duration_shorter_than_minutes: 1
permissions: [view, submit_answer, end_session, see_session_time, lock_down_as_exam_session]
if_in_facility: "cbtf"

@inducer inducer Feb 13, 2019

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

  • "CBTF" again is UIUC-specific terminology.

@inducer inducer Feb 13, 2019

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

  • Nothing wrong with adding this in a comment, but the generic test_center is probably better.
  • It would probably also be good to explain that these facility names are configured per-instance in local_settings.py.

if_session_duration_shorter_than_minutes: 45 # length until warning appears
permissions: [view, submit_answer, change_answer, end_session, see_correctness, see_session_time, lock_down_as_exam_session]

-
if_in_facility: test_center
if_in_facility: "cbtf"
message: "Your exam will end soon."
if_session_duration_shorter_than_minutes: 120 # keep in mind DRES students!

@inducer inducer Feb 13, 2019

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

  • DRES is UIUC-specific terminology (should use neutral phrasing)
  • Each participant record has an individual "time multiplier" that acts on this. I'm sympathetic to adding comments here to clarify this.

permissions: [view, submit_answer, change_answer, end_session, see_correctness, see_session_time, lock_down_as_exam_session]

-
if_in_facility: "cbtf"
if_in_progress: True
message: "You exam has ended. You may no longer make changes. Please click 'Submit assignment' to finish."
permissions: [view, end_session, see_correctness, see_session_time, lock_down_as_exam_session]
message: "Your exam has ended. You may no longer make changes to your answers. Please click 'Submit assignment' to finish."
permissions: [view, change_answer, end_session, see_correctness, see_correctness, see_session_time, lock_down_as_exam_session]

-
if_in_facility: test_center
if_in_facility: "cbtf"
permissions: [view, see_correctness, see_session_time]

-
Expand Down
3 changes: 3 additions & 0 deletions question-data/f.m
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
function [ y ] = f( x )
y = exp( -x .^ 2 );

@inducer inducer Feb 13, 2019

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This may have leaked from the Octave question PR.

end %function