Action #774

Feature #692: Perform an extensive interface review of all classes

Review sky module classes

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

Status:ClosedStart date:02/20/2013
Priority:NormalDue date:
Assigned To:Knödlseder Jürgen% Done:

100%

Category:-Estimated time:20.00 hours
Target version:00-08-00
Duration:

Recurrence

No recurrence.

History

#1 Updated by Knödlseder Jürgen over 11 years ago

The following classes have been reviewed:
Class Comment
GWcsAIT Creation
GWcsAZP Minor revision
GWcsCAR Minor revision
GWcsMER Minor revision
GWcsSTG Minor revision
GWcsTAN Minor revision
GHealpix Renamed from GWcsHPX
GSkyPixel Make class generic for 1D and 2D pixels
GSkymap Use generic 1D and 2D pixels, rename conversion methods consistently, revise interface
GSkyProjection Renamed GWcs to GSkyProjection
GWcs Renamed GWcslib to GWcs
GWcsRegistry Now only holds WCS projections
GSkyDir Kept as was

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

  • Status changed from New to In Progress
  • Assigned To set to Knödlseder Jürgen
  • % Done changed from 0 to 10

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

It should be checked whether GSkyPixel can be generalised to 1D and 2D pixelisations. This would simplify the interface for GSkymap and allow for a higher level ob abstraction. Generalisation can be achieved by setting for example the y index to -1 and using only the x index of 1D pixelisation.

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

  • % Done changed from 10 to 60

Knödlseder Jürgen wrote:

It should be checked whether GSkyPixel can be generalised to 1D and 2D pixelisations. This would simplify the interface for GSkymap and allow for a higher level ob abstraction. Generalisation can be achieved by setting for example the y index to -1 and using only the x index of 1D pixelisation.

GSkyPixel has been made generic. The class can now be used to access pixels in 1D and 2D pixelisations.

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

  • Status changed from In Progress to Closed
  • % Done changed from 60 to 100
  • Remaining (hours) changed from 20.0 to 0.0

Finished.

Also available in: Atom PDF