Skip to content

Conversation

@z-k-li
Copy link
Contributor

@z-k-li z-k-li commented Oct 8, 2025

Changes in this pull request

Added Siemens Quadra scanner in scanner.cxx and .h.

Testing performed

Built on own machine and test create_projdata_template.

Related issues

Checklist before requesting a review

  • I have performed a self-review of my code
  • [] I have added docstrings/doxygen in line with the guidance in the developer guide
  • [] I have implemented unit tests that cover any new or modified functionality (if applicable)
  • The code builds and runs on my machine
  • [] documentation/release_XXX.md has been updated with any functionality change (if applicable)

Contribution Notes

Please tick the following:

  • The content of this Pull Request (the Contribution) is intentionally submitted for inclusion in STIR (the Work) under the terms and conditions of the Apache-2.0 License.
  • I (or my institution) have signed the STIR Contribution License Agreement (not required for small changes).

@KrisThielemans KrisThielemans self-assigned this Oct 9, 2025
@KrisThielemans
Copy link
Collaborator

test_Scanner gives

WARNING: Scanner Siemens Quadra: inconsistent axial block info: 320 vs 323
Error. check_consistency

an extra ring between "Vision 600" scanners? Need to do something with virtual rings then.

@KrisThielemans
Copy link
Collaborator

FYI, discussion for future work (not this PR!) in #1388

@KrisThielemans
Copy link
Collaborator

To get this across the line, we need to change the num_axial_crystals_per_block to be a whole "Vision 600", and add an axial virtual crystal, as discussed. I'd be eager to merge this.

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