astropy / astropy/regions

Implement saving regions to ECSV via astropy Table

Open
#551 8 comments 0 reactions 0 assignees View on GitHub
API feature-request io needs-discussion
Dominant language
Python
Stars
65
Forks
64
Avg merge
2d 13h
Merged PRs (30d)
4

Description

I recently implemented IO to/from ECSV files via `astropy.table` in `specutils`, and coded up a quick minimal implementation to save out the spatial regions relevant to Jdaviz in https://github.com/spacetelescope/jdaviz/pull/2874 to provide export parity with spectral regions. I spoke to @larrybradley offline about upstreaming that code to here, and he pointed out that it would need to handle all the metadata available in `regions`, the other regions that we don't use in JDaviz, and implement a reader/parser as well. This issue is mostly to register my intent to do that at some point, and to provide a public place for any further discussion.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing the existing astropy.table ECSV implementation in specutils and the linked JDaviz PR #2874. Inventory the metadata supported by regions and the region types not used by JDaviz, then define reader and writer coverage with round-trip tests as the completion criteria.

Written by the indexing model from the issue text.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.