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

Remove parameter input option for what should be hardcoded Track "Operators"

    Details

      Description

      While testing graph track operations I noticed that there are several different "Log" operations:

      -"Log"
      -"Log10"
      -"Log2"
      -"Natural Log"

      A user may look at these available options and become confused as I was. If the base for Log10 is 10 and the base for Log2 is 2 and the base for the Natural Log is "e", what is the base for the plain "Log"?

      Further examination reveals that the plain "Log" uses whatever the default value is (its currently set to "0"). A better name for this operation may be "LogX" because it gives some indication to what the base is.

      Its also worth noting that these Log operations are available from the Graph tab. The problem here is that there can be confusing contradictions. For example, a user can select Log10 but they are still required to input a value for the base. So if the user selects Log10 but then puts 2 as the base, what should be the expected behavior? I'm sure you can see that this can cause a bit of confusion.

      I propose that we take one of two actions:

      1) Remove all the Log functions except for the plain "Log" and force the user to input their desired base when they use the operator. (This will remove confusion because the user will know exactly what the base is every time).

      OR

      2) Remove the ability to select a base when selecting "Log10", "Log2", or "Natural Log". The ability to input a base should only be allowed for the plain "Log" operator (which I vote to rename "LogX").

        Attachments

          Activity

          mason Mason Meyer (Inactive) created issue -
          dcnorris David Norris (Inactive) made changes -
          Field Original Value New Value
          Rank Ranked higher
          dcnorris David Norris (Inactive) made changes -
          Rank Ranked higher
          dcnorris David Norris (Inactive) made changes -
          Story Points 1
          ann.loraine Ann Loraine made changes -
          Epic Link IGBF-237 [ 15234 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          mason Mason Meyer (Inactive) made changes -
          Epic Link IGBF-237 [ 15234 ] IGBF-497 [ 15559 ]
          mason Mason Meyer (Inactive) made changes -
          Rank Ranked higher
          mason Mason Meyer (Inactive) made changes -
          Rank Ranked lower
          ann.loraine Ann Loraine made changes -
          Epic Link IGBF-497 [ 15559 ] IGBF-594 [ 15667 ]
          mason Mason Meyer (Inactive) made changes -
          Epic Link IGBF-594 [ 15667 ] IGBF-600 [ 15673 ]
          Hide
          mason Mason Meyer (Inactive) added a comment -

          From Ann:

          • The logarithmic functions log2, ln, and log10 should not have a “configure” option. I noticed that “sqrt” doesn’t have a “configure” option — so probably we can easily remove these for log2, ln, and log10?

          Show
          mason Mason Meyer (Inactive) added a comment - From Ann: • The logarithmic functions log2, ln, and log10 should not have a “configure” option. I noticed that “sqrt” doesn’t have a “configure” option — so probably we can easily remove these for log2, ln, and log10?
          mason Mason Meyer (Inactive) made changes -
          Sprint Sprint 19 [ 24 ]
          mason Mason Meyer (Inactive) made changes -
          Epic Link IGBF-600 [ 15673 ] IGBF-539 [ 15605 ]
          mason Mason Meyer (Inactive) made changes -
          Epic Link IGBF-539 [ 15605 ] IGBF-600 [ 15673 ]
          mason Mason Meyer (Inactive) made changes -
          Fix Version/s 8.3.3 Minor Release [ 10000 ]
          dcnorris David Norris (Inactive) made changes -
          Assignee David Norris [ dcnorris ]
          dcnorris David Norris (Inactive) made changes -
          Fix Version/s 8.4.0 Major Release [ 10001 ]
          Fix Version/s 8.3.3 Minor Release [ 10000 ]
          dcnorris David Norris (Inactive) made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          dcnorris David Norris (Inactive) made changes -
          Status In Progress [ 3 ] Needs Testing [ 10002 ]
          mason Mason Meyer (Inactive) made changes -
          Status Needs Testing [ 10002 ] Testing In Progress [ 10003 ]
          mason Mason Meyer (Inactive) made changes -
          Assignee David Norris [ dcnorris ] Mason Meyer [ mason ]
          mason Mason Meyer (Inactive) made changes -
          Fix Version/s 8.3.4 Minor Release [ 10000 ]
          Fix Version/s 8.4.0 Major Release [ 10001 ]
          dcnorris David Norris (Inactive) made changes -
          Summary Examine Redundancies and Contradictions in the "Log" Track Operators Remove parameter input option for what should be hardcoded Track "Operators"
          Hide
          mason Mason Meyer (Inactive) added a comment -

          My testing verifies that this issue is now functioning as expected. Since this story is resolved it will now be closed.

          Show
          mason Mason Meyer (Inactive) added a comment - My testing verifies that this issue is now functioning as expected. Since this story is resolved it 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 [ 15649 ] Fall 2019 Workflow Update [ 19346 ]
          ann.loraine Ann Loraine made changes -
          Workflow Fall 2019 Workflow Update [ 19346 ] Revised Fall 2019 Workflow Update [ 21463 ]

            People

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

              Dates

              • Created:
                Updated:
                Resolved: