Do not prefix camera metadata with label

[marktsuchida]

Fix the minority of cameras that use metadata keys "MyLabel-Key" instead of "Key" for some or all fields of the camera-generated metadata. In the case of TISCam, delete the redundant fields.

Affected cameras: Andor, AndorSDK3, PICAM, PrincetonInstruments, RaptorEPIX (& HoribaEPIX), TIScam, TwoPhoton.
PVCAM change is commented code only.

Fixes #831.

[marktsuchida]

Regarding the breaking change:

Given that devices get a default label equal to their name, one possibility is to add prefixes like Andor-, AndorSDK3-, etc., instead of just removing the prefixes outright. This has two advantages: it clarifies the data source and it opportunistically preserves compatibility for people using the default label. (PVCAM follows this style with PVCAM-FrameNr, etc., though I'm not aware of any other camera adapter doing so.)

Question would be whether to do this only for non-standard properties (I think yes).

[marktsuchida]

Ok, here's a better plan:

Andor

Raw key	Previous	New	Remarks
SRRFFrameTime-ms	label-SRRFFrameTime-ms	Andor-SRRFFrameTime	Custom tag
ElapsedTime-ms	label-ElapsedTime-ms	Andor-ElapsedTime-ms	Plain ElapsedTime-ms is NOT overwritten by Core, but is supposed to be ms since start of acquisition. But the value inserted here is the absolute MMTime ms value, so it should not use the standard key
ElapsedTime-ms(HW)	label-ElapsedTime-ms(HW)	Andor-ElapsedTime-ms(HW)	Custom tag
ImageNumber	label-ImageNumber	Andor-ImageNumber	Plain ImageNumber is currently overwritten by Core. The number produced here, however, may differ due to acquisition restart situations. Keep it separate.
Binning	label-Binning	Binning	Standard, not touched by Core. But the Binning property will produce label-Binning, which is often Andor-Binning, unless state cache metadata is disabled.
CurrentTemperature	label-CurrentTemperature	Andor-CurrentTemperature	Custom tag

AndorSDK3

The existing code prefixes the tags with Andor sCMOS Camera, not the label. So we can keep the exact behavior:

Full key (unchanged)
Andor sCMOS Camera-ImageNumber
Andor sCMOS Camera-ElapsedTime-ms
Andor sCMOS Camera-SRRFFrameTime-ms

PICAM

ElapsedTime-ms and ImageNumber. These do not offer anything that the unprefixed tags added by Core do not offer. Just remove.

PrincetonInstruments

This adapter is obsolete and not in any build. Should remove the whole adapter.

PVCAM

As stated above, only commented code is updated here.

RaptorEPIX, HoribaEPIX

All custom keys; prefix with RaptorEPIX- or HoribaEPIX-

TIScam

ElapsedTime-ms, ImageNumber, and Binning. The first 2 are redundant; remove. Keep Binning but without label prefix.

TwoPhoton

ElapsedTime-ms and ImageNumber. These might be sort of redundant, but there is value in setting them uniformly across the multiple channels of the camera. And ImageNumber is probably not generated correctly by Core for this multi-channel case with uniform camera label. Keep, with TwoPhoton- (rather than label) prefix.

CameraChannelIndex and CameraChannelName: these do in fact need to have the label prefix for backward compatibility, at least until we start handling these in the Core (#840). For now, add tags with and without the label prefix.