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

graph builder refactor for maintainability #62

Open
jahndan opened this issue Mar 1, 2024 · 0 comments
Open

graph builder refactor for maintainability #62

jahndan opened this issue Mar 1, 2024 · 0 comments
Assignees
Labels
wontfix This will not be worked on

Comments

@jahndan
Copy link
Collaborator

jahndan commented Mar 1, 2024

"a little functional programming never hurt anybody" but it's also much harder to onboard/maintain when it's incomprehensible

so basically this is a reminder that i should eventually rewrite graph builder using OOP instead for maintainability, although this is a fairly low priority right now as graph builder is currently working (and I'll be here to maintain it for a while) — because eventually someone else will have to deal with that code

also right now it takes multiple root courses and efficiently builds the full graph from them, but I want to rewrite (or at least, supplement) it in a way that allows for caching of graphs built from individual root courses, and then combining them after the fact - this should also play well with any serverless/edge functions we might want to use for this instead (if I maybe want to use python (wasm? 💀) for set-based graph building, once we get accurate and stabilized data)

@jahndan jahndan self-assigned this Mar 1, 2024
@jahndan jahndan added the documentation Improvements or additions to documentation label Mar 1, 2024
@jahndan jahndan added wontfix This will not be worked on and removed documentation Improvements or additions to documentation labels May 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

1 participant