basic template for hpge post processing

[tdixon97]

Draft PR that we can add to with the basic hpge post processing

• utils for IO
• basic windowing from evtid
• windowing from times
• basic detector response model (resolution)
• more flexible detector response
• dead layers (code is now in legend-hpges-pygeom
• generalisation to allow easy user customization

evt tier can be added next / later.
Related: legend-exp/legend-pygeom-hpges#51

[codecov]

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

[ManuelHu]

Please also add a (possibly empty) __init__.py to the hpge directory. Otherwise the automatic API docs are not built... (I also just found that out in #3)

[ManuelHu]

do not worry that all CI fails, this is a known problem and currently affects all our packages that (transitively) depend on pint for units. see hgrecco/pint#1969

[ManuelHu]

for me this PR looks quite fine. I did not really have a lot of time to use it, though.

There is certainly quite some code (mostly the utility things) that could already be shared with the optical part. But I would propose to unify them after merging your and also my latest changes - just to avoid large diffs to manually merge.

[gipert]

docs still need a lot of work and fixes but we can improve them in a future PR. you should rewrite the git history to remove the pre-commit bot commits before merging, I don't want to merge that junk. You just need to git rebase -i <hash of your first commit>, follow instructions to squash pre-commit bot commits into the previous, then force push

[tdixon97]

I think its worth taking time to cleanup and review things now. We could then review together in Munich.

…

________________________________ From: Luigi Pertoldi ***@***.***> Sent: Tuesday, November 19, 2024 11:47 PM To: legend-exp/reboost ***@***.***> Cc: Dixon, Toby ***@***.***>; Author ***@***.***> Subject: Re: [legend-exp/reboost] basic template for hpge post processing (PR #2) ⚠ Caution: External sender docs still need a lot of work and fixes but we can improve them in a future PR. you should rewrite the git history to remove the pre-commit bot commits before merging, I don't want to merge that junk. You just need to git rebase -i <hash of your first commit>, follow instructions to squash pre-commit bot commits into the previous, then force push — Reply to this email directly, view it on GitHub<#2 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ANSET43WLXD5OQTJABMTBRL2BO5XLAVCNFSM6AAAAABQK3HXK2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBWHEYDKNZUGQ>. You are receiving this because you authored the thread.Message ID: ***@***.***>

[gipert]

Yes, let's keep this open and merge it together.