[SyneRBI/SIRF] handle conveniences (PR #1390)

Casper da Costa-Luis

unread,

Mar 31, 2026, 7:03:04 AM (7 days ago) Mar 31

to SyneRBI/SIRF, Subscribed

Handle.new convenience method
functools.wraps backend functions in Handle.c*
for discussion: what is the purpose of (unused) check_stack/inspect.stack()?

Follow-up to #1388

You can view, comment on, or merge this pull request online at:

https://github.com/SyneRBI/SIRF/pull/1390

Commit Summary

4cc22e5 Handle.new convenience method
f1ed8df `functools.wraps` backend functions in `Handle.c*`
3f6f564 for discussion

File Changes

(5 files)

M src/Registration/pReg/Reg.py (57)
M src/common/SIRF.py (4)
M src/common/Utilities.py (12)
M src/xGadgetron/pGadgetron/Gadgetron.py (30)
M src/xSTIR/pSTIR/STIR.py (104)

Patch Links:

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

Casper da Costa-Luis

unread,

Mar 31, 2026, 8:48:22 AM (7 days ago) Mar 31

to SyneRBI/SIRF, Push

@casperdcl pushed 3 commits.

cccc629 HANDLE convenience singleton
ac8c6cd `functools.wraps` backend functions in `Handle.c*`
980e9b6 for discussion

—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.

Kris Thielemans

unread,

Mar 31, 2026, 7:26:14 PM (6 days ago) Mar 31

to SyneRBI/SIRF, Subscribed

@KrisThielemans approved this pull request.

ok, but now might be a good time to add a little bit of documentation on the Handle class (and indeed the new HANDLE. It is an implementation detail (should we call it _Handle in Python tradition?), but developers do need to know how this works :-)