tests of KeyError after positions calibration_elements are added to DB

[tobiaskleiner]

In export_array_elements_as_table in array_model.

[GernotMaier]

@tobiaskleiner - I guess you mean this function here?

simtools/simtools/model/array_model.py

Line 368 in f921472

def export_array_elements_as_table(self, coordinate_system="ground"):

(Always better to add a permanent link to issues.

Where exactly do you want to add a test for KeyError? Or is it already implemented and this issue is obsolete?

[tobiaskleiner]

@GernotMaier the current implementation doesn't export the coordinates of other array elements, only telescope model positions. Maybe we can generalize this.?
Then the KeyError test is not really needed anymore as there is the try ... except now.

[GernotMaier]

Yes, this should be generalised (possibly with the option of given a list of array element types in the call).

But do I see correctly the array_model only knows about telescopes as array elements, but about the calibration devices?

[tobiaskleiner]

Yes this needs to be added if we want to output all elements.