astropy / astropy/regions

Use "Shapely" or "sphere" for Astropy regions?

Open
#46 18 comments 0 reactions 0 assignees View on GitHub
question wishlist
Dominant language
Python
Stars
65
Forks
64
Avg merge
2d 13h
Merged PRs (30d)
4

Description

Should we use [Shapely](https://pypi.python.org/pypi/Shapely) and / or [Sphere](https://github.com/spacetelescope/sphere) and / or neither (i.e. re-implement or copy over things like polygon computations) as a dependency for this to-be `astropy.regions` package?

I don't remember if this important question was discussed in detail or decided at `PyAstro16`!?

Currently the API has `shapely` in many places, but nothing is really implemented and @astrofrog, I think it might just be the left-over of some API brainstorming a year or two ago?
https://github.com/astropy/regions/search?utf8=%E2%9C%93&q=shapely

@eteq @astrofrog @keflavich @larrybradley, all - thoughts?

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by searching the repository for the existing `shapely` references mentioned in the issue and review how they appear in the `astropy.regions` API. Then investigate the Shapely and Sphere dependency options and the prior discussion around PyAstro16. Done means reaching and recording a decision on whether to use either dependency or neither, with the API direction aligned to that decision.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.