Skip to content

Week 3 – clustering algorithms & report - #3

Merged
julianwuth merged 6 commits into
mainfrom
week-3
May 27, 2026
Merged

julianwuth merged 6 commits into
mainfrom
week-3

Conversation

@julianwuth

Copy link
Copy Markdown
Collaborator
  1. Implementation of the following clustering algorithms (with unit tests):
  • Gaussian Mixture Model

  • k-Means

  • HDBSCAN

    All deliver visualisation-friendly output for plotting within the GUI later.

  1. Scaffold and structure for report
  2. Continuous integration

Implementing algorithms to cluster tracks based on their features:

- Preprocessing of feature values
- Clustering techniques (Gaussian Mixed Models, k-Means and HDBSCAN)
- Visualisation friendly outputs
- Associated unit tests
@vandenman
vandenman requested review from LYang-uva and kastute21 May 25, 2026 12:02

@LYang-uva LYang-uva left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can the package be installed? -Yes
○ Is the vignette set up correctly? -Yes, the cells run in a logical order and the mock is installed before any feature call happens.
○ Did the unit tests set up with continuous integration run on GitHub? -yes
■ Did they pass? -yes
○ Anything else that comes to mind - Good job

@kastute21 kastute21 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can the package be installed? - yep, all good
Is the vignette set up correctly? - yes
Did the unit tests set up with continuous integration run on GitHub? - yes, and good amount of tests, the coverage checker is a nice touch
Did they pass? - all passed
Anything else that comes to mind - great job

@julianwuth
julianwuth merged commit 5be01a5 into main May 27, 2026
1 check passed
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