Skip to content
Snippets Groups Projects

Fix radec_to_xyz

Merged Marcel Loose requested to merge fix-radec_to_xyz into master

The radec_to_xyz() function contained an unnecessary conversion to AltAz coordinates, which also used a wrong LOFAR station location. In astropy ICRS coordinates can be converted to ITRS directly, without the need to go via AltAz. Fortunately, because the conversion to AltAz was just an unnecessary intermediate step, no real harm was done.

Merge request reports

Merge request pipeline #88522 passed

Merge request pipeline passed for 4f6f8da4

Merged by Marcel LooseMarcel Loose 7 months ago (Jul 19, 2024 9:29am UTC)

Merge details

  • Changes merged into with d1be70fb.
  • Deleted the source branch.

Pipeline #91524 failed

Pipeline failed for d1be70fb on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading