astropy / astropy/photutils

Make extract_stars recognize different conventions for x/y positions

Open
#966 1 comment 0 reactions 0 assignees View on GitHub
epsf_builder psf
Dominant language
Python
Stars
307
Forks
156
Avg merge
56m
Merged PRs (30d)
58

Description

I am using the ``DAOStarFinder`` to find stars and would like to use this table directly in ``extract_stars`` - it would be nice to be able to do this without having to recreate a new table or rename the columns to ``x``/``y``. Could ``extract_stars`` include some heuristics to recognize e.g. ``xcentroid``/``ycentroid``, as well as ``xcenter``/``ycenter`` and ``x_peak``/``y_peak``, which are all conventions used in photutils?

Contributor guide

Open the contributing guide

Research direction

Start by locating the extract_stars implementation and its tests, then inspect how DAOStarFinder exposes xcentroid/ycentroid. The change is done when extract_stars accepts xcentroid/ycentroid, xcenter/ycenter, and x_peak/y_peak conventions without requiring callers to recreate or rename the table, with tests covering these inputs.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.