motiwari / BanditPAM

BanditPAM C++ implementation and Python package
MIT License
647 stars 38 forks source link

Resolve conflicts in restructuring the script. Fixes #22 #107

Closed mailology closed 3 years ago

mailology commented 3 years ago

This is a new PR replacing the previous https://github.com/ThrunGroup/BanditPAM/pull/105. The fixes from the previous PR is applied on the latest main which solves the conflicts. Below are the fixes for restructuring the scripts:

I have tested the C++ and Python implementations of the code on MNIST-1k dataset. Both of them give expected results.

motiwari commented 3 years ago

Thanks @mailology ! I rebased this commit on top of main and merged it.