Skip to:

  1. Skip to Jira Navigation
  2. Skip to Side Navigation
  3. Skip to Main Content

Correct obs_lsst makeSkymap.py

Description

The parameters in obs_lsst/config/makeSkymap.py do not currently reproduce the lsst_cells_v1 skymap. This will be rectified by the addition of config.skyMap["rings"].tractBuilder.name = "cells". This ticket is to add that line to the above file.

Issue Matrix

hide

Activity

Show:
James Mullaney
January 20, 2025 at 7:09 PM
(edited)

Jenkins passed and branch merged to main.

James Mullaney
January 20, 2025 at 3:15 PM
(edited)

Thanks! I’m just doing a quick Jenkins run prior to merging to main.

Lee Kelvin
January 20, 2025 at 3:04 PM

I’ve tested this ticket here on our Princeton repo using:

After running, I now see:

From what I can tell, this looks ok to me - thanks for cleaning this up!

James Mullaney
January 20, 2025 at 2:01 PM

This is now ready for review.

After editding the config/makeSkyMap.py file to include the above extra line, I made my own private repo and registered the skymap in that:

Next, I checked to confirm that the new skymap is the same as the lsst_cells_v1 skymap:

Done
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Reviewers

Lee Kelvin

RubinTeam

Components

Checklist

Created January 17, 2025 at 10:18 PM
Updated January 20, 2025 at 7:10 PM
Resolved January 20, 2025 at 7:09 PM