Skip to content

Bug: Different classes matched #1

@lem0n4id

Description

@lem0n4id

Describe the bug

My friend and I have different classes yet it matched.

Specifically, it matched L49-BCSE103E-ELA-AB1-706-ALL and L46-BCSE103E-ELA-AB1-706-ALL

To Reproduce

Steps to reproduce the behavior:

  1. Paste L49-BCSE103E-ELA-AB1-706-ALL in text box 1
  2. Paste L46-BCSE103E-ELA-AB1-706-ALL in text box 2

Expected behavior

It shouldn't match because the slots(L49, L46) are different

Todo

Change the algorithm which matches the courses

Currently, the app matches courses by courseid-venue.

Since classes with the same venue and slot won't have different professors, we need to find the set of courses such that

  • courseid and venue matches
  • atleast one slot matches

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions