astronomy-commons / astronomy-commons/lsdb

HealpixPixel supports numpy arrays as inputs

Open
#346 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
55
Forks
26
Avg merge
4d 1h
Merged PRs (30d)
8

Description

Currently we use a single `HealpixPixel` object to represent one pixel, which means for multiple pixels we end up creating lists of objects. This is relatively slow, and there are multiple points in the code where we have to convert between a list of `HealpixPixels` to numpy array of orders and pixels. It would be good to support a single `HealpixPixel` object containing numpy arrays of orders and pixels to make this conversion a lot faster, and we can replace Lists of `HealpixPixel`s with a single object with arrays.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the HealpixPixel class and the code that converts lists of HealpixPixels to NumPy arrays of orders and pixels. Trace how single-pixel and multi-pixel inputs are used before defining the array-backed representation. Done means one HealpixPixel object can contain NumPy arrays and the relevant list-based conversions are replaced without changing behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
numpy, python
Domain
data-engineering
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.