Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-9342

histogram option update (input dialog and server support)

    XMLWordPrintable

    Details

      Description

      For uniform binning (fixed bin size), there are two different ways to give the input:
      1. number of bins (the current option only)
      OR
      2. bin width

      Label may needs to be changed as well.

      [March-2-2017]
      To have a choice of the number of bins or bin width, two radio buttons and one text box are added. By default, the number of bins is selected. When the radio button is selected, a new value either number of bins or bin width should be entered and validated. Then the histogram will be updated accordingly. Since no data range information is available before sending the request, the bin width can not be validated correction. Thus, an exception is thrown if the bin width is large than the data range.

      [March-16-17]
      Based on the modified requirements, the UI is

      1. Add two radio buttons and two input text boxes next to the two radio buttons, one of which is number of bins and the other is bin width.
      2. Add two input text boxes below the two set radio/text boxes. One of which is for min and the other is max

      After several discussions and iterations, we added the following:

      1. For single column histogram, since the data min/max is stored in the column variable array, they can be used to pre-fill the min and max.
      2. The bin width is calculated based on the number of bins and min and max and then the bin width is pre-filled.
      3. The bin width is recalculated each time when number of bins or min or max changes.
      4. When number of bin is selected, the bin width text box is disabled, when the bin width is selected, number of bins text box is disabled

        Attachments

          Activity

          Show
          zhang Lijun Zhang [X] (Inactive) added a comment - https://github.com/Caltech-IPAC/firefly/pull/335
          Hide
          zhang Lijun Zhang [X] (Inactive) added a comment -

          reviewed

          Show
          zhang Lijun Zhang [X] (Inactive) added a comment - reviewed
          Hide
          zhang Lijun Zhang [X] (Inactive) added a comment -

          merged

          Show
          zhang Lijun Zhang [X] (Inactive) added a comment - merged

            People

            Assignee:
            zhang Lijun Zhang [X] (Inactive)
            Reporter:
            xiuqin Xiuqin Wu [X] (Inactive)
            Reviewers:
            Emmanuel Joliet, Tatiana Goldina
            Watchers:
            Emmanuel Joliet, Lijun Zhang [X] (Inactive), Tatiana Goldina, Xiuqin Wu [X] (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.