Fix Map() repr to be more sane

[kosack]

The old repr was basically unreadable, as it printed the reprs of all sub-containers. This just changes the repr to be actually useful:

event.dl1.tel

After this change:

Map[int, ctapipe.containers.ArrayEventContainer](keys=[13, 6])

Before: Too much information was given. The Container reprs are only useful when dealing with a single one, not a list. What the user wants when looking at a Map is just the list of what is inside it.

Map(ctapipe.containers.DL1CameraContainer, {np.int16(5): ctapipe.containers.DL1CameraContainer:
                         image: Numpy array of camera image, after waveform
                                extraction.Shape: (n_pixel) if n_channels is 1
                                or data is gain selectedelse: (n_channels,
                                n_pixel) with default None
                     peak_time: Numpy array containing position of the peak of
                                the pulse as determined by the extractor.Shape:
                                (n_pixel) if n_channels is 1 or data is gain
                                selectedelse: (n_channels, n_pixel) with default
                                None
                    image_mask: Boolean numpy array where True means the pixel
                                has passed cleaning. Shape: (n_pixel, ) with
                                default None as a 1-D array with dtype bool
                      is_valid: True if image extraction succeeded, False if
                                failed or in the case of TwoPass methods, that
                                the first pass only was returned. with default
                                False
                  parameters.*: Image parameters with default None with type
                                <class
                                'ctapipe.containers.ImageParametersContainer'>, np.int16(11): ctapipe.containers.DL1CameraContainer:
                         image: Numpy array of camera image, after waveform
                                extraction.Shape: (n_pixel) if n_channels is 1
                                or data is gain selectedelse: (n_channels,
                                n_pixel) with default None
                     peak_time: Numpy array containing position of the peak of
                                the pulse as determined by the extractor.Shape:
                                (n_pixel) if n_channels is 1 or data is gain
                                selectedelse: (n_channels, n_pixel) with default
                                None
                    image_mask: Boolean numpy array where True means the pixel
                                has passed cleaning. Shape: (n_pixel, ) with
                                default None as a 1-D array with dtype bool
                      is_valid: True if image extraction succeeded, False if
                                failed or in the case of TwoPass methods, that
                                the first pass only was returned. with default
                                False
                  parameters.*: Image parameters with default None with type
                                <class
                                'ctapipe.containers.ImageParametersContainer'>})

[maxnoe]

Map(ctapipe.containers.DL1CameraContainer: [np.int16(5), np.int16(11)])

I find it a bit weird to see value_type: keys. Usually maps are the other way around.

Maybe something like: Map[key_type, value_type](keys={keys}) would be more intuitive?

[ctao-sonarqube]

Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

[mexanick]

I actually found the previous verbose repr useful :D But I see the point in making it concise.

[maxnoe]

I actually found the previous verbose repr useful :D But I see the point in making it concise.

It woudl repeat the same container repr for every entry in the map... that's hardly useful.

I proposed to change the container repr to show the data content and not the help, but @kosack opposed this. I still think a data containers repr should show its content and not its docstring.