Misleading attribute "userValidSignal" in Pyramid class

[joao-aveiro]

Description

When the Pyramid class is instantiated with a non null userValidSignal argument, if self.postProcessing="slopesMaps" the initialization will set self.validI4Q with the provided array (L400). This could be misleading since the class attribute self.validSignal is then set with the concatenation of self.validI4Q with itself, which will result in self.validSignal having double the size if the user provided what is supposed to be the final valid signal array.

Fix

Possibly set self.validSignal = self.userValidSignal, thus replicating the logic used for self.postProcessing="fullFrame". If it is of interest to maintain the possibility of the user providing what effectively is self.validI4Q , one option would be to try to infer which case it is based on the intensity maps (and possibly also validate the shape/dimension of user-provided mask).