Opened 12 years ago

Closed 12 years ago

Last modified 11 years ago

#170 closed enhancement (fixed)

Refactor FITS writing code

Reported by: MatthewWhiting Owned by: MatthewWhiting
Priority: normal Milestone: Release-1.3
Component: Output Version: 1.2.2
Severity: normal Keywords:
Cc:

Description

The code to save mask, moment map, recon, smoothed etc arrays to FITS files has a bit of repeated code.

It may be useful to refactor into a class-based structure, along the lines of what we did with the reading code (see #166).

Change History (4)

comment:1 Changed 12 years ago by MatthewWhiting

Status: newassigned

See #105 - want to include baseline array as well here.

comment:2 Changed 12 years ago by MatthewWhiting

Version: 1.2.11.2.2

Starting this in [1115],[1116]. Idea is to have a base class that holds all necesary members and does basic creation, header writing and closing, with derived classes for each type of array that do specific header writing and writing of data.

The code should all go in FitsIO at the end (currently in Cubes).

Will also need a Cube method to handle the calling of each of them - this will simplify the interface as well (ie. from mainDuchamp.cc's perspective).

comment:3 Changed 12 years ago by MatthewWhiting

Resolution: fixed
Status: assignedclosed

Done and implemented in [1120],[1121]. Closing ticket.

comment:4 Changed 11 years ago by MatthewWhiting

Milestone: Release-2.0Release-1.3
Note: See TracTickets for help on using tickets.