Uploaded image for project: 'DRP 2-D Pipeline'
  1. DRP 2-D Pipeline
  2. PIPE2D-338

Handle lack of pfsConfig files for the LAM data

    Details

    • Type: Task
    • Status: Done (View Workflow)
    • Priority: Normal
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: 6.0
    • Component/s: None
    • Labels:
      None
    • Story Points:
      2
    • Sprint:
      2019 B

      Description

      LAM data are (of course) not coming with pfsConfig file that are needed for the full pipeline to run. At the moment to solve this, I am taking dummy pfsConfig files from drp_stella_data and renaming them to match what my script (modelled after pfs_integration_test.sh) is asking for. This ticket is to either 1. automatize this process or 2. expand the documentation with the swindle that I am doing, described above

        Attachments

          Issue Links

            Activity

            Hide
            price price added a comment -

            I've got ingestPfsImages.py working now so that it if it can't find the pfsConfig file, it will look for the pfiDesign file and use that to create the pfsConfig file, and then ingest it. So I believe this is ready to merge, once people are happy with it.

            Show
            price price added a comment - I've got ingestPfsImages.py working now so that it if it can't find the pfsConfig file, it will look for the pfiDesign file and use that to create the pfsConfig file, and then ingest it. So I believe this is ready to merge, once people are happy with it.
            Hide
            arnaud.lefur arnaud.lefur added a comment -

            Just a quick comment : There is no 'red' bundle in the dcb and we can at maximum connect 5 fiber bundles at the same time.

            so having fiber [2, 3, 308, 339, 340, 342, 649, 650] together is not possible i'm afraid.

            Show
            arnaud.lefur arnaud.lefur added a comment - Just a quick comment : There is no 'red' bundle in the dcb and we can at maximum connect 5 fiber bundles at the same time. so having fiber [2, 3, 308, 339, 340, 342, 649, 650] together is not possible i'm afraid.
            Hide
            price price added a comment -

            Thanks for letting me know. I've removed the red setting.

            Show
            price price added a comment - Thanks for letting me know. I've removed the red setting.
            Hide
            hassan hassan added a comment -

            Following discussion with @rhl: will proceed with merging branch in order to make progress. Will address subsequent problems that could arise in future tickets.

            Show
            hassan hassan added a comment - Following discussion with @rhl: will proceed with merging branch in order to make progress. Will address subsequent problems that could arise in future tickets.
            Hide
            price price added a comment -

            Merged to master.

            Show
            price price added a comment - Merged to master.

              People

              • Assignee:
                price price
                Reporter:
                ncaplar ncaplar
                Reviewers:
                ncaplar
              • Votes:
                0 Vote for this issue
                Watchers:
                Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: