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

Parse prerequisites into tokens #5

Open
GabrielDTB opened this issue Jun 28, 2023 · 3 comments
Open

Parse prerequisites into tokens #5

GabrielDTB opened this issue Jun 28, 2023 · 3 comments
Labels
backend Something that doesn't directly impact the user #4 very hard Large changes that require thinking

Comments

@GabrielDTB
Copy link
Owner

Prerequisites are written by humans (I think?) and so quite a number of them are irregular, making their parsing into tokens that can be manipulated for various purposes quite a challenge. I don't think it's possible to write a set of rules that encapsulates all of the classes' prerequisites perfectly, but I do believe it's possible to correctly parse ~98% of them. The rest can have manual rules added.

This would allow prerequisite trees and simplifying prerequisites into a single, standard, unambiguous format.

@GabrielDTB GabrielDTB added the #4 very hard Large changes that require thinking label Jun 28, 2023
@GabrielDTB
Copy link
Owner Author

There is previous work done on this up to f791c49, which is the commit before I began refactoring.

@GabrielDTB
Copy link
Owner Author

Might be easier to look into gaining access to and using the workday api instead of parsing these from the catalog

@GabrielDTB
Copy link
Owner Author

Another angle is to write grammar on a per-department basis, since grammar is pretty consistent within departments and only starts to differ when you cross over.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Something that doesn't directly impact the user #4 very hard Large changes that require thinking
Projects
None yet
Development

No branches or pull requests

1 participant