Skip to content

Add proton CT runnable example#895

Draft
acoussat wants to merge 3 commits intoOpenGATE:masterfrom
acoussat:protonct-example
Draft

Add proton CT runnable example#895
acoussat wants to merge 3 commits intoOpenGATE:masterfrom
acoussat:protonct-example

Conversation

@acoussat
Copy link
Copy Markdown
Contributor

@acoussat acoussat commented Feb 4, 2026

Proton CT runnable example. The goal is to use it in connection with PCT in order to have a complete proton CT pipeline (from MC simulation to tomographic reconstruction) easily runnable.

The example is taken from PCT, from which it will probably be removed. This example was already somewhat available in GATE in the form of a GATE 9 to GATE 10 migration guide.

@acoussat acoussat marked this pull request as draft February 5, 2026 09:22
@acoussat
Copy link
Copy Markdown
Contributor Author

acoussat commented Feb 5, 2026

Before merging, I'd like to finish PCT's "sibling" PR, just to make sure that everything works alright between the two software.

@SimonRit
Copy link
Copy Markdown

SimonRit commented Feb 5, 2026

Before merging, I'd like to finish PCT's "sibling" PR, just to make sure that everything works alright between the two software.

We should reflect on what we will do in the future with PCT. There was a (dysfunctional) test using protonct.py which I suggest to remove
RTKConsortium/PCT@6659355

Could you discuss with @axel-grc what you want to do with PCT tests in the future? If you want to use protonct.py, it's not clear to me that it should be in the Gate repository, a link in the documentation to PCT is sufficient.

@acoussat
Copy link
Copy Markdown
Contributor Author

acoussat commented Feb 5, 2026

I discussed yesterday with @tbaudier and @axel-grc about how to reorganize the code, here are my plans:

  • create a proper proton CT example on the GATE side, as GATE already had an example but only in the form of a GATE 9 to GATE 10 conversion guide (https://opengate-python.readthedocs.io/en/10.0.3/user_guide/user_guide_how_to_convert_example_1.html) (done in this PR)
  • still on the GATE side, make this proton CT example callable, which makes it easy for users to quickly generate proton CT data (done in this PR)
  • create a Python example on PCT side which calls this new GATE script to generate data, then reconstruct it
  • have this "end-to-end" example automatically called during PCT CI, which allows to check if PCT is compatible with the current GATE version
  • include this example in PCT documentation, as in RTK

The PCT side is almost ready. Additionally, I finished updating AddTrackerUncertainty.py, for which @nkrah thinks it would make sense to have directly in GATE, but I leave that for a second round of PRs.

Of course let me know if you disagree with this plan, nothing is set in stone!

@SimonRit
Copy link
Copy Markdown

SimonRit commented Feb 5, 2026

Sounds good. The callable proton CT will be part of the wheel I suppose.
I don't think AddTrackerUncertainty.py should be part of GATE but we can rediscuss it with @nkrah.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants