Tamas Vami

@tomeichlersmith you practically implemented this, is there any reason why you didnt PR it? Should I do it?

As a validation I ran the config at 400, the hard brem seem to stop there and that's where the daughters are starting: @tylerhoroho-UVA I put the file that contains...

In https://github.com/LDMX-Software/ldmx-sw/pull/1289/commits/24f9e0ba0a0b7a15b4ca80ae642c25c9a8447974?diff=split&w=1 I updated the DQM plots to show the particle types, now it looks like this:

OK I'm done with the filter that doesnt require the photon to come from the target. The changes for the file size: the brem photon is tagged, no other brem...

OK in the last push I introduced `require_photon_fromTarget_` that governs if we want to force the photon to come from the target. Worked nicely on one event, I'll make a...

Interesting, the `volume.contains("target")` actually brings in much more than just the target. @tylerhoroho-UVA I assume this is not good. Maybe we could enforce instead that it's coming from around Z=0...

Yes, I think everything the the target.gdml, i.e. these https://github.com/LDMX-Software/ldmx-sw/blob/trunk/Detectors/data/ldmx-det-v14-8gev/target.gdml#L72-L91 are going to be called target by G4. OK, let's do it with the `-0.5 < z < 0.5`, I'll...

Talked to Tyler about the issue above, and we concluded that it's better to keep this as it is (in real data we wont know if the interaction happened on...

> I would encourage you to survey values of the biasing threshold and the sorting threshold to see how high you can raise it before losing access to the events...

> we were comparing v12 and v14 of the ECal Yeah sure that's a good example where this would be needed. But we are done with those comparisons, right? What's...