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

Continue QuantumGraph implementation

    XMLWordPrintable

    Details

      Description

      This is a continuation of work started in DM-14334. Main goal of this ticket is to extend preflight solver with joins between skymap units and camera units so that we have more or less complete solution for building quantum graphs. 

        Attachments

          Activity

          Hide
          pschella Pim Schellart [X] (Inactive) added a comment -

          Feel free to merge now, I'll rebase DM-15096 instead.

          Show
          pschella Pim Schellart [X] (Inactive) added a comment - Feel free to merge now, I'll rebase DM-15096 instead.
          Hide
          jbosch Jim Bosch added a comment -

          Comments on the PR.

          Show
          jbosch Jim Bosch added a comment - Comments on the PR.
          Hide
          salnikov Andy Salnikov added a comment -

          Jim Bosch, thanks for review! Sorry, I forgot to push pipe_supertask commit, just made a PR for that too. It only adds an example SuperTask that I used for testing my pre-flight changes. You can check it too if you prefer.

          Show
          salnikov Andy Salnikov added a comment - Jim Bosch , thanks for review! Sorry, I forgot to push pipe_supertask commit, just made a PR for that too. It only adds an example SuperTask that I used for testing my pre-flight changes. You can check it too if you prefer.
          Hide
          jbosch Jim Bosch added a comment -

          Changes in pipe_supertask look fine; I left one comment on the PR, but it's really an unrelated question about preflight in general prompted by a bit of code there.

          Show
          jbosch Jim Bosch added a comment - Changes in pipe_supertask look fine; I left one comment on the PR, but it's really an unrelated question about preflight in general prompted by a bit of code there.
          Hide
          salnikov Andy Salnikov added a comment -

          Thanks for review! Merged both packages to master.

          Show
          salnikov Andy Salnikov added a comment - Thanks for review! Merged both packages to master.

            People

            Assignee:
            salnikov Andy Salnikov
            Reporter:
            vaikunth Vaikunth Thukral
            Reviewers:
            Jim Bosch
            Watchers:
            Andy Salnikov, Fritz Mueller, Jim Bosch, Pim Schellart [X] (Inactive), Vaikunth Thukral
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                CI Builds

                No builds found.