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

dev portfolio submission #3 #413

Closed
wants to merge 2 commits into from
Closed

dev portfolio submission #3 #413

wants to merge 2 commits into from

Conversation

syeboah10
Copy link

So far still working on the Recommendations for CUReviews

As of right now, loaded all courses from FA23 into clusters
Totaling 49 total clusters

Currently we have some outliers in the clusters, so some clusters may only have a few classes in them.
In order to fix this issue im looking to merge cluster that dont meet the length requirement, for each thing inside the cluster, we find another cluster that they are most similar too, and that is its new home.

Current Goal:
Make each cluster be of 50 classes at the lowest
load clusters to db

Here are some images of current progress
Screen Shot 2023-10-22 at 9 57 41 PM
Screen Shot 2023-10-22 at 9 31 30 PM

Additional Google docs for the design docs for the system
https://docs.google.com/document/d/1j3EWao56qGPDSXOMQO0rVS16LL2GbVhQ4sFxA36vde0/edit

@syeboah10 syeboah10 requested a review from a team as a code owner October 23, 2023 02:03
@dti-github-bot
Copy link
Member

[diff-counting] Significant lines: 14.

@wizhaaa
Copy link
Collaborator

wizhaaa commented Oct 25, 2023

Would it be possible to copy and paste your Python script into a new file just so we can and reference the script

@wizhaaa wizhaaa closed this May 1, 2024
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 this pull request may close these issues.

3 participants