Change request #1829

GSkyProjection::coordsys should return "CEL" and not "EQU"

Added by Knödlseder Jürgen over 7 years ago. Updated almost 4 years ago.

Status:ClosedStart date:07/30/2016
Priority:NormalDue date:
Assigned To:Knödlseder Jürgen% Done:

100%

Category:-
Target version:1.7.0
Duration:

Description

Just discovered that the GSkyProjection::coordsys method returns “EQU” instead of “CEL”, but all ctools accept “CEL”.

I don’t remember why I have chosen “EQU”. Should investigate this, and unless there is a reason, change to “CEL”.


Recurrence

No recurrence.

History

#1 Updated by Knödlseder Jürgen about 7 years ago

  • Target version changed from 1.2.0 to 1.3.0

#2 Updated by Knödlseder Jürgen almost 7 years ago

  • Target version deleted (1.3.0)

#3 Updated by Knödlseder Jürgen almost 5 years ago

  • Target version set to 1.7.0

#4 Updated by Knödlseder Jürgen almost 4 years ago

  • Status changed from New to Pull request
  • % Done changed from 0 to 100

I changed GSkyProjection::coordsys() so that it now returns CEL instead of EQU.

EQU is everywhere else also replaced by CEL, yet the GSkyProjection::coordsys(const std::string& coordsys) method still accepts EQU on input for compatibility reasons.

#5 Updated by Knödlseder Jürgen almost 4 years ago

  • Status changed from Pull request to Closed

Merged into devel.

Also available in: Atom PDF