Skip to content
Snippets Groups Projects

WS-2355 Calibration wrester telescope

Merged Daniel Nemergut requested to merge ws2355-calibration_curation into 2.8.4-DEVELOPMENT
All threads resolved!

Getting the telescope from a calibration's exec block.

ALMA doesn't have an exec block, so this is assuming the telescope is ALMA rather than also querying by the alma_ous_id because that seems like overkill.

Merge request reports

Merge request pipeline #15790 passed

Merge request pipeline passed for 5493b390

Approval is optional

Merged by Daniel NemergutDaniel Nemergut 10 months ago (May 28, 2024 5:34pm UTC)

Merge details

  • Changes merged into 2.8.3.1-patch with 7dd25188.
  • Deleted the source branch.

Pipeline #15817 passed

Pipeline passed for 7dd25188 on 2.8.4-DEVELOPMENT

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Sam Kagan resolved all threads

    resolved all threads

  • Sam Kagan
  • added 1 commit

    • c7489eeb - Removed cal join in the first statement and moved all the logic to the second...

    Compare with previous version

  • Daniel Nemergut resolved all threads

    resolved all threads

  • Pushed an update that removed the cal join in the first statement and does a query for the telescope from the EB for both ALMA and others in a single union query.

    Tested the second query with both ALMA and EVLA cal SPLs and they return a single telescope name (the correct one!)

  • Sam Kagan
  • Sam Kagan
  • added 1 commit

    Compare with previous version

  • Sam Kagan approved this merge request

    approved this merge request

  • Sam Kagan resolved all threads

    resolved all threads

  • Charlotte Hausman approved this merge request

    approved this merge request

  • mentioned in commit 7dd25188

  • Please register or sign in to reply
    Loading