Prototype code for focal plane stuff

[Paul Price]

I will attach prototype code. It does not all meet the requirements in the
document (I made some changes based on my experience), but it comes close, and I
hope it is useful.

[Paul Price]

fpa.tar.gz

[Paul Price]

This is the psModule prototype code that I mentioned at the meeting.

[Paul Price]

I'm in the process of updating this code to conform to the SDRS. Please let me
know if you will be working on it actively.

[David.Robbins@…]

We are planning on folding this code in this week. Let me know when the
updates are finished so that I can proceed. Thanks.

[Paul Price]

fpa.tar.bz2

[Paul Price]

Note that the pmFPAConstruct in attachment 65 uses an old method for determining
the source of the BIASSEC and TRIMSEC (a "CELL.BIASSEC.SOURCE" keyword is in the
SDRS, but not implemented here yet) which needs to be updated.

[Paul Price]

pmConfigRead calls psLogSetDestination with a value of 0, which turns off
logging, instead of directing it to stdout, as intended. See bug 508.

[Paul Price]

Found a bug in pmFPAConstruct: each cell needs its own copy of the cell metadata
("concepts"), otherwise it can't add its own name.

Solution is to change getCellData:

Read data for a particular cell from the camera configuration
static psMetadata *getCellData(const psMetadata *camera, The camera configuration

const char *cellName The name of the cell

)

{

bool status = true; Result of MD lookup
psMetadata *cells = psMetadataLookupMD(&status, camera, "CELLS"); The CELLS
if (! status) {

psError(PS_ERR_IO, false, "Unable to determine CELLS of camera.\n");
return NULL;

}

psMetadata *cellData = psMetadataLookupMD(&status, cells, cellName); The

data for the particular cell

if (! status) {

psError(PS_ERR_IO, false, "Unable to find specs for cell %s: ignored\n", cellName);

}

Need to create a new instance, so that each cell can work with its own
psMetadata *copy = psMetadataAlloc();
psMetadataIterator *iter = psMetadataIteratorAlloc(cellData, PS_LIST_HEAD,

NULL); Iterator

psMetadataItem *item = NULL; Item from iteration
while (item = psMetadataGetAndIncrement(iter)) {

if (item->type == PS_META_MULTI

item->type == PS_META_META) {

psLogMsg(func, PS_LOG_WARN, "PS_META_MULTI and PS_META_META are not

supported in a cell "

"definition --- %s ignored.\n", item->name);

continue;

}
if (! psMetadataAdd(copy, PS_LIST_TAIL, item->name, item->type, item->comment,

item->data.V)) {

psAbort(func, "Should never reach here!\n");

}

}
psFree(iter);

return copy;

}

Then, to remove memory leaks, there should be a psFree(cellData) after every
psCellAlloc.

[Paul Price]

fpa.tar.gz

[Paul Price]

Oh, I also updated pmFPAConstruct to implement the CELL.BIASSEC.SOURCE stuff.

Also added prototypes for the concept lookups.

[Paul Price]

Reworked this to do the concept stuff at ingest, as is in the latest version of
the SDRS.

[Paul Price]

fpa.tar.gz

[Paul Price]

Sorry, I'm playing with the definitions again (pmReadout and the p_pmHDU).

[gusciora@…]

It's not quite fixed (this was prototype code, not a bug). The code should be
in the current CVS tree.

[Paul Price]

The code has evolved some since the last installment recorded here. Yesterday
(14 Dec 2005) I branched psModule (pap_branch_051214). I will plug in my
current code and tidy up.