Readability of larndsim

[YifanC]

I think one of the reason larndsim readability is low is that when we define intermediate data, we uses shape of previous intermediate data, for example here (instead of using len(unique_pix), we use pixels_signals.shape[0])

Suggestions:

1. Add comments
2. Explicitly define the intermediate data type
3. Enforce consistency check