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

Create DMTNs for this epic

    XMLWordPrintable

    Details

    • Story Points:
      10
    • Epic Link:
    • Sprint:
      AP S18-6
    • Team:
      Alert Production

      Description

      Create DMTN documents chronicling the work done for this epic and knowledge gained about the Optimistic Pattern Matcher algorithm.

      Deliverables: 1-3 DMTNs:
      Document on the prototype python OPMb matcher.
      Document describing the data and reasons the algorithm failures.
      Document fully describing the final algorithm implemented in the stack.

        Attachments

          Activity

          No builds found.
          cmorrison Chris Morrison [X] (Inactive) created issue -
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Field Original Value New Value
          Epic Link DM-7370 [ 26456 ]
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Epic Link DM-7370 [ 26456 ] DM-7366 [ 26452 ]
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Sprint Alert Production F16 - 11 [ 289 ]
          Hide
          cmorrison Chris Morrison [X] (Inactive) added a comment -

          Cloned DMTN repository.

          Show
          cmorrison Chris Morrison [X] (Inactive) added a comment - Cloned DMTN repository.
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Status To Do [ 10001 ] In Progress [ 3 ]
          krughoff Simon Krughoff made changes -
          Sprint Alert Production F16 - 11 [ 289 ] Alert Production F16 - 11, Alert Production F16 - 11b [ 289, 295 ]
          krughoff Simon Krughoff made changes -
          Rank Ranked higher
          krughoff Simon Krughoff made changes -
          Sprint Alert Production F16 - 11, Alert Production F16 - 11b [ 289, 295 ] Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c [ 289, 295, 296 ]
          krughoff Simon Krughoff made changes -
          Sprint Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c [ 289, 295, 296 ] Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c, Alert Production S17 - 12 [ 289, 295, 296, 305 ]
          krughoff Simon Krughoff made changes -
          Rank Ranked higher
          krughoff Simon Krughoff made changes -
          Sprint Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c, Alert Production S17 - 12 [ 289, 295, 296, 305 ] Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c, Alert Production S17 - 12, Alert Production S17 - 1 [ 289, 295, 296, 305, 355 ]
          krughoff Simon Krughoff made changes -
          Rank Ranked higher
          krughoff Simon Krughoff made changes -
          Sprint Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c, Alert Production S17 - 12, Alert Production S17 - 1 [ 289, 295, 296, 305, 355 ] Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c, Alert Production S17 - 12, Alert Production S17 - 1, Alert Production S17 - 2 [ 289, 295, 296, 305, 355, 361 ]
          krughoff Simon Krughoff made changes -
          Rank Ranked higher
          krughoff Simon Krughoff made changes -
          Sprint Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c, Alert Production S17 - 12, Alert Production S17 - 1, Alert Production S17 - 2 [ 289, 295, 296, 305, 355, 361 ] Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c, Alert Production S17 - 12, Alert Production S17 - 1, Alert Production S17 - 2, Alert Production S17 - 3 [ 289, 295, 296, 305, 355, 361, 605 ]
          krughoff Simon Krughoff made changes -
          Rank Ranked higher
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Sprint Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c, Alert Production S17 - 12, Alert Production S17 - 1, Alert Production S17 - 2, Alert Production S17 - 3 [ 289, 295, 296, 305, 355, 361, 605 ] Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c, Alert Production S17 - 12, Alert Production S17 - 1, Alert Production S17 - 2 [ 289, 295, 296, 305, 355, 361 ]
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Status In Progress [ 3 ] To Do [ 10001 ]
          krughoff Simon Krughoff made changes -
          Epic Link DM-7366 [ 26452 ] DM-10399 [ 32121 ]
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Status To Do [ 10001 ] In Progress [ 3 ]
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Sprint Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c, Alert Production S17 - 12, Alert Production S17 - 1, Alert Production S17 - 2 [ 289, 295, 296, 305, 355, 361 ] Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c, Alert Production S17 - 12, Alert Production S17 - 1, Alert Production S17 - 2, AP S18-3 [ 289, 295, 296, 305, 355, 361, 683 ]
          swinbank John Swinbank made changes -
          Sprint Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c, Alert Production S17 - 12, Alert Production S17 - 1, Alert Production S17 - 2, AP S18-3 [ 289, 295, 296, 305, 355, 361, 683 ] Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c, Alert Production S17 - 12, Alert Production S17 - 1, Alert Production S17 - 2, AP S18-3, AP S18-4 [ 289, 295, 296, 305, 355, 361, 683, 684 ]
          swinbank John Swinbank made changes -
          Sprint Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c, Alert Production S17 - 12, Alert Production S17 - 1, Alert Production S17 - 2, AP S18-3, AP S18-4 [ 289, 295, 296, 305, 355, 361, 683, 684 ] Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c, Alert Production S17 - 12, Alert Production S17 - 1, Alert Production S17 - 2, AP S18-3, AP S18-4, AP S18-5 [ 289, 295, 296, 305, 355, 361, 683, 684, 685 ]
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Reviewers John Swinbank [ swinbank ]
          Status In Progress [ 3 ] In Review [ 10004 ]
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Risk Score 0
          Hide
          swinbank John Swinbank added a comment -

          Just to make sure I'm reviewing the right thing — the text is what currently appears at https://dmtn-031.lsst.io, right? (It's marked as “Latest Revision: 2018-02-21”, but I think that's just because the machinery needs some adjustment.)

          Show
          swinbank John Swinbank added a comment - Just to make sure I'm reviewing the right thing — the text is what currently appears at https://dmtn-031.lsst.io , right? (It's marked as “Latest Revision: 2018-02-21”, but I think that's just because the machinery needs some adjustment.)
          Hide
          cmorrison Chris Morrison [X] (Inactive) added a comment -

          I would say just look at the rst on github. I didn't even know the io page was updating at all let alone when it will update after pushing the latest version.

          Show
          cmorrison Chris Morrison [X] (Inactive) added a comment - I would say just look at the rst on github. I didn't even know the io page was updating at all let alone when it will update after pushing the latest version.
          Hide
          tjenness Tim Jenness added a comment -

          https://dmtn-031.lsst.io/v/index.html indicates it was updated today.

          The copyright needs updating (it says 2016).

          Show
          tjenness Tim Jenness added a comment - https://dmtn-031.lsst.io/v/index.html indicates it was updated today. The copyright needs updating (it says 2016).
          Hide
          swinbank John Swinbank added a comment -

          Since there wasn't a outstanding PR, I've made a bunch of tweaks to the text and added a series of comments (as “notes”). These are now available on this PR against master.

          Please do read the PR and take my formatting/typo/copy-editing fixes as suggestions rather than requirements! The things in “note” blocks are what I'd really like you to address.

          Overall, this is very nice, though. I've made a few suggestions for clarification of the wording, but the material seems comprehensive and well thought through. I like it!

          Show
          swinbank John Swinbank added a comment - Since there wasn't a outstanding PR, I've made a bunch of tweaks to the text and added a series of comments (as “notes”). These are now available on this PR against master . Please do read the PR and take my formatting/typo/copy-editing fixes as suggestions rather than requirements! The things in “note” blocks are what I'd really like you to address. Overall, this is very nice, though. I've made a few suggestions for clarification of the wording, but the material seems comprehensive and well thought through. I like it!
          swinbank John Swinbank made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]
          swinbank John Swinbank made changes -
          Status Reviewed [ 10101 ] In Review [ 10004 ]
          krzys Krzysztof Findeisen made changes -
          Sprint Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c, Alert Production S17 - 12, Alert Production S17 - 1, Alert Production S17 - 2, AP S18-3, AP S18-4, AP S18-5 [ 289, 295, 296, 305, 355, 361, 683, 684, 685 ] AP S18-6 [ 686 ]
          Hide
          swinbank John Swinbank added a comment -

          Thanks for your responses to my comments!

          I am now happy with almost everything. There are two comments outstanding which I'm still not clear about — could you take a look? (I think maybe you already explained one to me in person, but I can't reconstruct it from the text, which I think means the text still needs a bit of tweaking).

          Apart from those two, you should probably cast an eye over my edits at https://github.com/lsst-dm/dmtn-031/pull/1/commits/ea1c5e759a12263612139c7001f922c9958a503b just to make sure that none of my editing altered your meaning or introduced any errors.

          Then I think we're good to go...

          Show
          swinbank John Swinbank added a comment - Thanks for your responses to my comments! I am now happy with almost everything. There are two comments outstanding which I'm still not clear about — could you take a look? (I think maybe you already explained one to me in person, but I can't reconstruct it from the text, which I think means the text still needs a bit of tweaking). Apart from those two, you should probably cast an eye over my edits at https://github.com/lsst-dm/dmtn-031/pull/1/commits/ea1c5e759a12263612139c7001f922c9958a503b just to make sure that none of my editing altered your meaning or introduced any errors. Then I think we're good to go...
          Hide
          cmorrison Chris Morrison [X] (Inactive) added a comment -

          Merged u/swinbank/edit edit's into main text.

          Show
          cmorrison Chris Morrison [X] (Inactive) added a comment - Merged u/swinbank/edit edit's into main text.
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Resolution Done [ 10000 ]
          Status In Review [ 10004 ] Done [ 10002 ]

            People

            Assignee:
            cmorrison Chris Morrison [X] (Inactive)
            Reporter:
            cmorrison Chris Morrison [X] (Inactive)
            Reviewers:
            John Swinbank
            Watchers:
            Chris Morrison [X] (Inactive), John Swinbank, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.