search

astropy / regions

Astropy package for region handling
https://astropy-regions.readthedocs.io
BSD 3-Clause "New" or "Revised" License
61 stars 55 forks source link

Merge spectral regions #228

Open keflavich opened 5 years ago

keflavich commented 5 years ago

Spectral-regions have been defined in specutils: https://github.com/astropy/specutils/pull/289

They should be included in regions eventually, especially since CRTF now supports spectral region serialization.

cdeil commented 5 years ago

@keflavich - What does including spectral regions here mean?

Is it just adding something to the I/O and specutils becomes an optional dependency?

Or do you mean changes to the data classes and possibly backwards-incompat breaks?

keflavich commented 5 years ago

It means adding things; I don't see any reason there should be any backward-incompatible breaks.

I'm uncertain whether specutils would become a dependency.