Skip to content

SIMPA Definition of MSOT inVision system has the wrong radius #126

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
tomelse opened this issue Apr 18, 2022 · 3 comments · Fixed by #130
Closed

SIMPA Definition of MSOT inVision system has the wrong radius #126

tomelse opened this issue Apr 18, 2022 · 3 comments · Fixed by #130

Comments

@tomelse
Copy link

tomelse commented Apr 18, 2022

Link here to code
The correct radius for the MSOT inVision should be 0.0405 m as far as I know. This is the value specified in the metadata from the study acquisitions anyway.
@kdreher
Copy link
Collaborator

kdreher commented Apr 19, 2022

I think we originally modelled it after this paper in which the radius is 40 mm. However, the pitch seems to be 0.735 mm instead of 0.74 mm. So there is definitely still something wrong. Maybe they updated the device with another radius or pitch.
We should ask iThera directly to clarify that.

@jgroehl
Copy link
Collaborator

jgroehl commented Apr 19, 2022

I'm pretty sure I modeled it based on a technical drawing I received from iThera.
I'll look into it again!

jgroehl added a commit that referenced this issue Apr 27, 2022
@jgroehl
Update InVision device parameters
96eb425 
Slightly updated parameters based on feedback of iThera staff.
Fixes #126
@jgroehl jgroehl mentioned this issue Apr 27, 2022
Merged
@jgroehl
Copy link
Collaborator

jgroehl commented Apr 27, 2022

We contacted iThera and they sent us the details again. Indeed, the pitch is 0.735 instead of 0.74, but the radius seems correct!

@jgroehl jgroehl closed this as completed Apr 27, 2022
jeremieglt pushed a commit to jeremieglt/simpa that referenced this issue Mar 5, 2025
@jgroehl
Update InVision device parameters
f86a89b 
Slightly updated parameters based on feedback of iThera staff.
Fixes IMSY-DKFZ#126
jeremieglt pushed a commit to jeremieglt/simpa that referenced this issue Mar 6, 2025
@jgroehl
Update InVision device parameters
40d2f5b 
Slightly updated parameters based on feedback of iThera staff.
Fixes IMSY-DKFZ#126
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update InVision device parameters IMSY-DKFZ/simpa
3 participants
@jgroehl @kdreher @tomelse