Commons Math
  1. Commons Math
  2. MATH-1059

Use FastMath instead of Math within CM

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.2
    • Fix Version/s: 3.3
    • Labels:
      None

      Description

      Some code in CM still uses Math.xxx instead of the counterparts in FastMath. This could lead to subtle differences with different jvms as could be seen in MATH-1057.

      All calls to Math shall be replaced by calls to FastMath.

        Activity

        Hide
        Thomas Neidhart added a comment -

        Finished changes in r1547633.

        Remaining uses of Math:

        • PI
        • ulp
        • for FastMath validation and performance tests
        Show
        Thomas Neidhart added a comment - Finished changes in r1547633. Remaining uses of Math: PI ulp for FastMath validation and performance tests
        Hide
        Luc Maisonobe added a comment -

        Closing all resolved issue now available in released 3.3 version.

        Show
        Luc Maisonobe added a comment - Closing all resolved issue now available in released 3.3 version.

          People

          • Assignee:
            Unassigned
            Reporter:
            Thomas Neidhart
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development