Uploaded image for project: 'Machine Learning Library'
  1. Machine Learning Library
  2. ML-272

Minor improvements to Concord

    XMLWordPrintable

Details

    • Improvement
    • Status: Open
    • Minor
    • Resolution: Unresolved
    • None
    • None
    • None
    • None

    Description

      Concord issues
      ML/PopulationEstimate.ecl, line 23 (old_oe1 definition): substracting a matrix from itself is a poor way to create a zero matrix. Just use 1 cell of zero.

      ML/PopulationEstimate, line 27 (out definition): The MAP function is a better choice than nested IF functions.

      ML/PopulationEstimate.ecl, line 64 (result1 and result2 definitions): It would be better to use the Mat subtraction and ABS function: MAX( Mat.Sub(zz1,zz2), ABS(Value) );

      ML/Tests/Validation/Concord.ecl line 1, change "is" to "in" on comment.

      ML/PopulationEstimate.ecl, line 1. "Estimate" is misspelled.

      Attachments

        Activity

          People

            anybody Available for anyone
            johnholt John Holt
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated: