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

Refactor solver module #214

Open
mattjones315 opened this issue Aug 17, 2023 · 0 comments
Open

Refactor solver module #214

mattjones315 opened this issue Aug 17, 2023 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@mattjones315
Copy link
Collaborator

Per a recent conversation in issue #205, we've determined that our recent software development efforts have outstripped the implementation of the DistanceSolver class. Namely, it's become apparent that the inheritance structure is limiting our ability to cleanly implement new distance-based algorithms and perhaps makes things unneccessarily complex. A move to composition-based software has been proposed.

The purpose of this issue is to discuss & plan a refactor of the DistanceSolver class and perhaps the entire solver module. In doing so, it would be ideal to maintain the current API as much as possible as to not break any existing code already relying on the solver module.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants