Tamas Vami

Results 386 comments of Tamas Vami

I did what I could, I have no idea how to do the light pipes, SiPMs, and support structure so somebody else will have to do that

Could be false-postive? @EinarElen wdyt?

Yes it is compiler, but I noticed it in the ASAN build

hi @therwig did you write this class? I also tried to add some of it in the logging system and get rid of the Printouts in https://github.com/LDMX-Software/ldmx-sw/pull/1800 but it lead...

I think if you do ``` just configure-asan-ubsan just build just fire .github/validation_samples/ecal_pn/config.py ``` the "potential out-of-bounds" should come up! Altho I did test that recently. What I tried to...

I made a PR where it's configurable and only the positive is used https://github.com/LDMX-Software/ldmx-sw/pull/1886

@tomeichlersmith maybe you can add more about the "Describe the solution you'd like"?

> if we should have a validation workflow for geometry changes? I agree we need this, but I think we should have this after the sensitive volumes are defined, then...

I'd like to point out that 1) This doesnt validate the geometry (we cant until https://github.com/LDMX-Software/ldmx-sw/issues/1533 is resolved) 2) This does check that passive element part, but then how different...