Uploaded image for project: 'IGB'
  1. IGB
  2. IGBF-394

The "Sqrt" graph track operation is performing the "Power" operation rather than Square Root

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      Description

      *Version noticed: 8.3.0

      While testing graph track operations I noticed that the "Sqrt" operation is not functioning properly.

      Instead of taking the Square Root of the values it raises the values to a power of 2.

      Also, it adds the words "Power - " to the beginning of the track title undergoing the operation (a clear sign that the Sqrt operation is off).

      It seems that this "Sqrt" function has been implemented to "Square" the values rather than finding the "SquareRoot" as the abbreviation indicates.

        Attachments

          Issue Links

            Activity

            mason Mason Meyer (Inactive) created issue -
            mason Mason Meyer (Inactive) made changes -
            Field Original Value New Value
            Description *Version noticed: 8.3.0

            While testing graph track operations I noticed that the "Sqrt" operation is not functioning properly.

            Instead of taking the Square Root of the values it raises the values to a power of 2.

            Also, it adds the words "Power - " to the beginning of the track title undergoing the operation (a clear sign that the Sqrt operation is off).
            *Version noticed: 8.3.0

            While testing graph track operations I noticed that the "Sqrt" operation is not functioning properly.

            Instead of taking the Square Root of the values it raises the values to a power of 2.

            Also, it adds the words "Power - " to the beginning of the track title undergoing the operation (a clear sign that the Sqrt operation is off).

            It seems that this "Sqrt" function has been implemented to "Square" the values rather than finding the "SquareRoot" as the abbreviation indicates.
            dcnorris David Norris (Inactive) made changes -
            Rank Ranked higher
            dcnorris David Norris (Inactive) made changes -
            Sprint Sprint 14 [ 19 ]
            dcnorris David Norris (Inactive) made changes -
            Rank Ranked lower
            dcnorris David Norris (Inactive) made changes -
            Story Points 0.25
            dcnorris David Norris (Inactive) made changes -
            Assignee Tarun Mall [ tmall ]
            dcnorris David Norris (Inactive) made changes -
            Rank Ranked higher
            dcnorris David Norris (Inactive) made changes -
            Sprint Sprint 14 [ 19 ]
            dcnorris David Norris (Inactive) made changes -
            Rank Ranked higher
            ann.loraine Ann Loraine made changes -
            Epic Link IGBF-439 [ 15497 ]
            dcnorris David Norris (Inactive) made changes -
            Sprint Sprint 15 [ 20 ]
            dcnorris David Norris (Inactive) made changes -
            Rank Ranked higher
            tmall Tarun Mall (Inactive) made changes -
            Status Open [ 1 ] Pull Request Submitted [ 10101 ]
            dcnorris David Norris (Inactive) made changes -
            Status Pull Request Submitted [ 10101 ] Needs Testing [ 10002 ]
            mason Mason Meyer (Inactive) made changes -
            Status Needs Testing [ 10002 ] Testing In Progress [ 10003 ]
            mason Mason Meyer (Inactive) made changes -
            Assignee Tarun Mall [ tmall ] Mason Meyer [ mason ]
            Hide
            mason Mason Meyer (Inactive) added a comment -

            My testing verifies that the "Sqrt" operation is now functioning properly after changes to the code. Since this story is resolved it will now be closed.

            Show
            mason Mason Meyer (Inactive) added a comment - My testing verifies that the "Sqrt" operation is now functioning properly after changes to the code. Since this story is resolved it will now be closed.
            Hide
            tmall Tarun Mall (Inactive) added a comment -

            commit e7e7de74c0b188b9daf08a416385a62da17c63e2
            Author: Tarun K. Mall <c2tarun@hotmail.com>
            Date: Tue Mar 10 14:39:13 2015 -0400

            Sqrt now calculating square root

            Show
            tmall Tarun Mall (Inactive) added a comment - commit e7e7de74c0b188b9daf08a416385a62da17c63e2 Author: Tarun K. Mall <c2tarun@hotmail.com> Date: Tue Mar 10 14:39:13 2015 -0400 Sqrt now calculating square root
            mason Mason Meyer (Inactive) made changes -
            Resolution Done [ 10000 ]
            Status Testing In Progress [ 10003 ] Closed [ 6 ]
            mason Mason Meyer (Inactive) made changes -
            Resolution Done [ 10000 ]
            Status Closed [ 6 ] Open [ 1 ]
            mason Mason Meyer (Inactive) made changes -
            Assignee Mason Meyer [ mason ] Tarun Mall [ tmall ]
            Hide
            mason Mason Meyer (Inactive) added a comment -

            If someone tries to take the Sqrt of a graph containing negative numbers then math says the sqrt should be "i" (an imaginary number). This, however, can't be drawn on a graph. What IGB currently does is draw to negative infinity. Instead, the sqrt of negative portions of graphs should just draw as zero. (It would be nice to have a way to show discontinuity of the graph, but this is not currently possible with IGB's current graph implementation.)

            Show
            mason Mason Meyer (Inactive) added a comment - If someone tries to take the Sqrt of a graph containing negative numbers then math says the sqrt should be "i" (an imaginary number). This, however, can't be drawn on a graph. What IGB currently does is draw to negative infinity. Instead, the sqrt of negative portions of graphs should just draw as zero. (It would be nice to have a way to show discontinuity of the graph, but this is not currently possible with IGB's current graph implementation.)
            mason Mason Meyer (Inactive) made changes -
            Link This issue relates to IGBF-491 [ IGBF-491 ]
            dcnorris David Norris (Inactive) made changes -
            Sprint Sprint 15 [ 20 ]
            dcnorris David Norris (Inactive) made changes -
            Sprint Sprint 16 [ 21 ]
            dcnorris David Norris (Inactive) made changes -
            Rank Ranked higher
            mason Mason Meyer (Inactive) made changes -
            Epic Link IGBF-439 [ 15497 ] IGBF-480 [ 15542 ]
            dcnorris David Norris (Inactive) made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            dcnorris David Norris (Inactive) made changes -
            Rank Ranked higher
            dcnorris David Norris (Inactive) made changes -
            Rank Ranked higher
            tmall Tarun Mall (Inactive) made changes -
            Status In Progress [ 3 ] Needs Testing [ 10002 ]
            mason Mason Meyer (Inactive) made changes -
            Assignee Tarun Mall [ tmall ] Mason Meyer [ mason ]
            mason Mason Meyer (Inactive) made changes -
            Epic Link IGBF-480 [ 15542 ] IGBF-497 [ 15559 ]
            mason Mason Meyer (Inactive) made changes -
            Status Needs Testing [ 10002 ] Testing In Progress [ 10003 ]
            Hide
            mason Mason Meyer (Inactive) added a comment -

            It seems that the graphs being drawn to negative infinity is no longer a reproducible behavior. Possibly, code refactoring may have corrected this. Since this operation is now functioning as expected the story will now be closed.

            Show
            mason Mason Meyer (Inactive) added a comment - It seems that the graphs being drawn to negative infinity is no longer a reproducible behavior. Possibly, code refactoring may have corrected this. Since this operation is now functioning as expected the story will now be closed.
            mason Mason Meyer (Inactive) made changes -
            Resolution Done [ 10000 ]
            Status Testing In Progress [ 10003 ] Closed [ 6 ]
            ann.loraine Ann Loraine made changes -
            Workflow Loraine Lab Workflow [ 15650 ] Fall 2019 Workflow Update [ 19347 ]
            ann.loraine Ann Loraine made changes -
            Workflow Fall 2019 Workflow Update [ 19347 ] Revised Fall 2019 Workflow Update [ 21464 ]

              People

              • Assignee:
                mason Mason Meyer (Inactive)
                Reporter:
                mason Mason Meyer (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: