Uploaded image for project: 'DataFu'
  1. DataFu
  2. DATAFU-42

Simplify BagGroup output

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.3.0
    • Labels:
      None

      Description

      BagGroup keeps the redundant group information in its output.
      E.g., see DATAFU-38:

      (1,{(b,1),(a,2)},{(B,{(B,3)}),(A,{(A,1),(A,2)})})
      (2,{(c,1),(b,2)},{(B,{(B,3),(B,5)}),(A,{(A,1),(A,2)}),(C,{(C,4),(C,6)})})
      

      can be

      (1,{(b,1),(a,2)},{(B,{3}),(A,{1,2})})
      (2,{(c,1),(b,2)},{(B,{3,5}),(A,{1,2}),(C,{4,6})})
      

      without loss of information

      Given that the bug DATAFU-38 rendered this function quite useless and it was fixed just last week, I think BagGroup has not been used before, so this backward-incompatible change will not break any existing code.

        Attachments

        1. DATAFU-42.patch
          46 kB
          Matthew Hayes
        2. DATAFU-42-v2.patch
          46 kB
          Matthew Hayes

          Activity

            People

            • Assignee:
              mhayes Matthew Hayes
              Reporter:
              sds Sam Steingold
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: