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

Merge generic and specialized exchange methods #37

Closed
m-Py opened this issue Sep 25, 2019 · 0 comments
Closed

Merge generic and specialized exchange methods #37

m-Py opened this issue Sep 25, 2019 · 0 comments
Labels
cleanup wontfix This will not be worked on

Comments

@m-Py
Copy link
Owner

m-Py commented Sep 25, 2019

Right now I have three functions that implement an exchange algorithm; 2x specialized functions that are speed optimized for maximizing the kmeans and cluster editing objectives, respectively, 1x a generic version that can maximize any objective function.

This means there is a lot of redundant code. It would be desirable to merge the three functions.

The difficulty in merging is that each of the three functions has need for different data structures that are generated and updated throughout the exchange method. I need to test if it is possible to merge the functions in a reasonable way despite this difficulty.

@m-Py m-Py added the cleanup label Sep 25, 2019
@m-Py m-Py added the wontfix This will not be worked on label Jan 29, 2020
@m-Py m-Py closed this as completed Jan 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

1 participant