Metadata is overwritten by consecutive reconstructions

[ziw-liu]

The recOrder reconstruction writes its settings into FOV-level .zattrs. However the entire setting object is dumped, so the next channel will overwrite the settings, leaving only the last one.

[ziw-liu]

recOrder/recOrder/cli/apply_inverse_transfer_function.py

Line 270 in 57f611e

output_dataset.zattrs["settings"] = settings.dict()

[talonchandler]

Thanks @ziw-liu.

Do you think something like this is the right move?
output_dataset.zattrs[f"settings-{output_channel_name}"] = settings.dict()

cc @edyoshikun

[ziw-liu]

Thanks @ziw-liu.

Do you think something like this is the right move? output_dataset.zattrs[f"settings-{output_channel_name}"] = settings.dict()

cc @edyoshikun

I think this is also a chance to use namespacing, as the generic 'settings' field can collide with other things. So something like:

"recOrder-napari":{
    "channel 1": {},
    "channel 2": {},
}