[SyneRBI/SIRF] hkem example needs cgitb (causing ctest failure) (Issue #1395)

1 view
Skip to first unread message

Kris Thielemans

unread,
Apr 20, 2026, 7:52:50 PMApr 20
to SyneRBI/SIRF, Subscribed
KrisThielemans created an issue (SyneRBI/SIRF#1395)

https://github.com/SyneRBI/SIRF/blob/7cef6cb2fafe15e2f18a55ebf04eb9fb7728d5df/examples/Python/PET/hkem_reconstruction.py#L42
This is not documented though, and it isn't installed in any of our instructions, nor https://github.com/SyneRBI/SIRF-SuperBuild/blob/master/docker/requirements.yml

This file is run via the PET_DEMOS_PYTHON ctest.

Moreover, cgitb seems deprecated, see https://anaconda.org/channels/conda-forge/packages/legacy-cgi/overview

Can we just remove this line?


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <SyneRBI/SIRF/issues/1395@github.com>

Edoardo Pasca

unread,
Apr 21, 2026, 8:34:18 AMApr 21
to SyneRBI/SIRF, Subscribed
paskino left a comment (SyneRBI/SIRF#1395)

It seems small is never used in the file. Suggestion is to remove the line.


Reply to this email directly, view it on GitHub, or unsubscribe.

You are receiving this because you are subscribed to this thread.Message ID: <SyneRBI/SIRF/issues/1395/4288551033@github.com>

Evgueni Ovtchinnikov

unread,
May 5, 2026, 9:10:12 AM (3 days ago) May 5
to SyneRBI/SIRF, Subscribed

Closed #1395 as completed via 724d102.


Reply to this email directly, view it on GitHub, or unsubscribe.

Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.Message ID: <SyneRBI/SIRF/issue/1395/issue_event/25177094172@github.com>

Reply all
Reply to author
Forward
0 new messages