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

algorithm #7

Open
yuuwe-n opened this issue Aug 10, 2022 · 2 comments · Fixed by #24
Open

algorithm #7

yuuwe-n opened this issue Aug 10, 2022 · 2 comments · Fixed by #24
Assignees
Labels
functionality basic functionality

Comments

@yuuwe-n
Copy link
Collaborator

yuuwe-n commented Aug 10, 2022

The general education requirement will be found, the algo will find best teachers for each requirement. The algo will take into account the weight of others. The goal is to find best courses/teachers for a student. The weight will based off for now the rating of the teacher (number of ratings may affect it later on).

Plans for using dijkstra's algorithm to find teachers in the same department and find courses in pathway with best weight. Each graph will contain a department's courses, each node containing the teacher and the connections are the weights. The nodes that connect with the most weight, will be returned for that GE requirement.

I am not sure why we cant just get the GE requirement, and return the teacher with the highest rating in that course requirement. Since courses fuffill many GE requirements, isnt it gonna be kind of messy.

@yuuwe-n yuuwe-n added the functionality basic functionality label Aug 10, 2022
@yuuwe-n yuuwe-n added this to the mvp milestone Aug 10, 2022
@yuuwe-n
Copy link
Collaborator Author

yuuwe-n commented Aug 11, 2022

algo is for series of classes, map for each series of classes
hopefully saves time instead of iterating for each professor for highest rating

for one off classes, we just find highest rating

@yuuwe-n
Copy link
Collaborator Author

yuuwe-n commented Aug 12, 2022

change to model, we will have this a WIP, for later, and have this for when we get GE requirements/major requirements working

we could not get GE requirements because of PDF permission issues.

@yuuwe-n yuuwe-n removed this from the 0.1 MVP milestone Aug 12, 2022
@yuuwe-n yuuwe-n linked a pull request Aug 12, 2022 that will close this issue
@yuuwe-n yuuwe-n reopened this Aug 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
functionality basic functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants