Skip to content
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.

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

validate curricula on requisite constrains #126

Closed
haydenfree opened this issue Oct 1, 2021 · 0 comments · Fixed by #127
Closed

validate curricula on requisite constrains #126

haydenfree opened this issue Oct 1, 2021 · 0 comments · Fixed by #127
Assignees

Comments

@haydenfree
Copy link
Member

There should be a test on requisite contrains. Extend isvalid_curriculum to handle this.

Example:

    Course A   --- PREREQ --> Course C
      |                          |
     COREQ                   STRICT-CO
      |                          |
      V                          V
    Course B                   Course B

Course A lists Course B as a corequisite (may be taken prior to or at the same time as Course A).
Course A is a prerequisite of Course C (Course A must be taken before Course C)
Course B is listed as a strict corequisite of Course C (MUST be taken at the same time as Course C)
These three cannot all be true. This locks Course B to be taken after Course A, but that violates the first corequisite constraint.

@haydenfree haydenfree linked a pull request Oct 7, 2021 that will close this issue
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 a pull request may close this issue.

2 participants