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

Move general math related classes to separate library #152

Closed
cicirello opened this issue Sep 17, 2021 · 0 comments · Fixed by #155
Closed

Move general math related classes to separate library #152

cicirello opened this issue Sep 17, 2021 · 0 comments · Fixed by #155

Comments

@cicirello
Copy link
Owner

Summary
Package org.cicirello.math, and all of its subpackages, contain classes related to various math concepts (e.g., basic statistics, linear algebra, etc). None of it is actually needed or used (with the exception of the random number generation classes of package org.cicirello.math.rand) by the core functionality of the JPT library (i.e., by the permutation distance metrics, sequence distance metrics, etc). Most of this was included in this library in support of the sample programs.

There is another issue (#150) related to creating a library for the random number generation classes. Either include the rest of the org.cicirello.math hierarchy in that proposed library, or extract it to a different new library.

The benefit to a different new library is that JPT will need the random number lib as a dependency, but not the rest of it. The benefit to just rolling this into #150 is that there would be a single new library to maintain just with slightly broader scope (math rather than random number algs).

Either way this will be a breaking change, so schedule this with release 3.0.0. Less severe of a break if integrated into #150 since that lib will become a dependency of JPT 3.0.0.

@cicirello cicirello added this to the Release 3.0.0 milestone Sep 17, 2021
@cicirello cicirello self-assigned this Sep 17, 2021
@cicirello cicirello added this to To do in Computational Intelligence Research via automation Sep 17, 2021
@cicirello cicirello mentioned this issue Sep 23, 2021
10 tasks
cicirello added a commit that referenced this issue Sep 24, 2021
Computational Intelligence Research automation moved this from To do to Done Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

1 participant