You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ignite.apache.org by "Oleg Ignatenko (JIRA)" <ji...@apache.org> on 2018/07/30 13:13:00 UTC

[jira] [Created] (IGNITE-9124) Remove some dead code in math.exceptions and optimization packages of ML module

Oleg Ignatenko created IGNITE-9124:
--------------------------------------

             Summary: Remove some dead code in math.exceptions and optimization packages of ML module
                 Key: IGNITE-9124
                 URL: https://issues.apache.org/jira/browse/IGNITE-9124
             Project: Ignite
          Issue Type: Task
          Components: ml
    Affects Versions: 2.5
            Reporter: Oleg Ignatenko


Doing code coverage analysis discovered a bit of unused code:

# in package [math.exceptions|https://github.com/apache/ignite/tree/master/modules/ml/src/main/java/org/apache/ignite/ml/math/exceptions] - classes ConvergenceException / MathIllegalStateException, InsufficientDataException, MathArithmeticException, MathIllegalNumberException / NumberIsTooSmallException, NonPositiveDefiniteMatrixException, NonSymmetricMatrixException, NullArgumentException, SingularMatrixException, UnknownProviderException. In very early releases of ML these exceptions were used by some linear algebra code. This code has been very deeply reworked since but outdated exceptions were left hanging unattended, making rather confusing impression of the current API.
# in [optimization|https://github.com/apache/ignite/tree/master/modules/ml/src/main/java/org/apache/ignite/ml/optimization] - interface Updater and two classes implementing it: BarzilaiBorweinUpdater and SimpleUpdater. This code has left hanging after cleanup that was made for release 2.5 (IGNITE-8232). Per my discussion with [~dmitrievanthony] (who most actively maintains this package) it would be better to remove this code.

This task is to remove code mentioned above.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)