Skip to content

rename _wcs to low_level_wcs#73

Closed
eteq wants to merge 28 commits intoastrofrog:low-level-wcsfrom
eteq:low-level-wcs
Closed

rename _wcs to low_level_wcs#73
eteq wants to merge 28 commits intoastrofrog:low-level-wcsfrom
eteq:low-level-wcs

Conversation

@eteq
Copy link

@eteq eteq commented Aug 2, 2018

APE14 specifically says it has to have this name. This is important because the high level object is what now will be in things like nddata, and users will want to be able to access the low-level object without accessing the private variable.

astrofrog and others added 28 commits July 28, 2018 22:14
@astrofrog astrofrog force-pushed the low-level-wcs branch 2 times, most recently from dfcf5c2 to d36b96c Compare September 28, 2018 11:13
@astrofrog
Copy link
Owner

This should now be done in the main PR

@astrofrog astrofrog closed this Sep 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants