Skip to content

Comments

Add detector-centers.py#64

Merged
hombit merged 2 commits intomainfrom
detector-center
Jan 20, 2026
Merged

Add detector-centers.py#64
hombit merged 2 commits intomainfrom
detector-center

Conversation

@hombit
Copy link
Collaborator

@hombit hombit commented Jan 20, 2026

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a new Python script that generates a Parquet file containing LSSTCam detector center coordinates in both field angle and focal plane coordinate systems.

Changes:

  • Adds a new script detector-centers.py that extracts detector geometry information from LSSTCam using the LSST stack
  • Creates a Parquet file with detector IDs and their corresponding center coordinates in field angle and focal plane systems

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@hombit hombit enabled auto-merge January 20, 2026 22:09
@codecov
Copy link

codecov bot commented Jan 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.66%. Comparing base (8a16dc4) to head (5479be8).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #64   +/-   ##
=======================================
  Coverage   65.66%   65.66%           
=======================================
  Files          23       23           
  Lines        1203     1203           
=======================================
  Hits          790      790           
  Misses        413      413           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hombit hombit merged commit ee4ce9e into main Jan 20, 2026
7 checks passed
@hombit hombit deleted the detector-center branch January 20, 2026 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant