Replies: 9 comments 6 replies
-
|
Hi, Are you on Windows or Linux? Does |
Beta Was this translation helpful? Give feedback.
-
|
Hi,
I am on Windows and projector_oct.oct exists in the source folder!
Best regards,
Tom
…On Fri, Jan 28, 2022 at 1:27 PM villekf ***@***.***> wrote:
Hi,
Are you on Windows or Linux? Does projector_oct.oct exist in the source
folder?
—
Reply to this email directly, view it on GitHub
<#31 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB3Y2EHPYVB6HAXGXONXMWTUYKDUJANCNFSM5NARNKUQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
|
As a clarification, are you running the |
Beta Was this translation helpful? Give feedback.
-
|
Hi,
The path was set and I am running it on its own but taking care that all
"option" parameters are initalised/defined.
Best regards,
Tom
…On Fri, Jan 28, 2022 at 2:37 PM villekf ***@***.***> wrote:
As a clarification, are you running the lor_pixel_count_prepass.m on its
own or part of e.g. reconstructions_main? Have you added the
source-folder to the Octave path?
—
Reply to this email directly, view it on GitHub
<#31 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB3Y2EBFZRPHLKHXUJIJF5TUYKLYBANCNFSM5NARNKUQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
|
Does it work if you run it while inside the source-folder? |
Beta Was this translation helpful? Give feedback.
-
|
What exactly do you mean by "inside the folder"? How would you suggest
trying to run the " [ lor, lor_orth, lor_vol] = projector_oct( Ny, Nx,
Nz,......"
…On Fri, Jan 28, 2022 at 3:08 PM villekf ***@***.***> wrote:
Does it work if you run it while inside the source-folder?
—
Reply to this email directly, view it on GitHub
<#31 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB3Y2EETHJGFB2X2OBZBTXTUYKPNTANCNFSM5NARNKUQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
|
What I mean is that the source-folder of OMEGA would be the current directory in Octave when you run the function. |
Beta Was this translation helpful? Give feedback.
-
|
Hi,
Yes, it is, but unfortunately, the error is still the same. I will test the
code on another Windows system with the new Matlab.
Being able to run the code under Octave would be perfect, but it might be
that I did something wrong also during the installation process.
Thank you very much for your prompt reply and efforts. I will try to
discover what might be causing the problem and let you know if I find out
the reason, hopefully after the weekend when I will have some more time to
devote to the problem.
Best regards,
Tom
…On Fri, Jan 28, 2022 at 3:47 PM villekf ***@***.***> wrote:
What I mean is that the source-folder of OMEGA would be the current
directory in Octave when you run the function.
—
Reply to this email directly, view it on GitHub
<#31 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB3Y2EFBDPKZWATO36OUOYDUYKT7FANCNFSM5NARNKUQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
|
Just tested this myself and after some debugging, I believe I've found the reason. The previous update that introduced CT support, also broke implementation 1 for PET in Octave. This is because in |
Beta Was this translation helpful? Give feedback.
-
|
Hi,
Thank you for your efforts and explanation. I will try the suggested change
during the day.
Best regards,
Tom
…On Mon, Jan 31, 2022 at 5:12 PM villekf ***@***.***> wrote:
Just tested this myself and after some debugging, I believe I've found the
reason. The previous update that introduced CT support, also broke
implementation 1 for PET in Octave. This is because in projector_oct.cpp
the function name is projector_octCT. If you don't need CT support at
all, simply modify DEFUN_DLD(projector_octCT, prhs, nargout,
"projector_octCT") { to DEFUN_DLD(projector_oct, prhs, nargout,
"projector_oct") { and it should work after compiling the code again (I
haven't managed to test this yet though). I'll fix this in master tomorrow.
Testing after the update was, unfortunately, lacking on Octave with regards
to PET.
—
Reply to this email directly, view it on GitHub
<#31 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB3Y2ECUUKMOXX2Y3VSPVIDUY2YFZANCNFSM5NARNKUQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
|
This has now been fixed on master. I was able to successfully compute the reconstructions for implementation 1, though this should have affected implementation 4 and the Thank you for reporting this issue. I'll have to try to test the Octave execution better in the future. |
Beta Was this translation helpful? Give feedback.
-
|
HELLO. ASCII -> ROOT Then I get the following error. Beginning LOR precomputation phase. This may take several minutes. thank you |
Beta Was this translation helpful? Give feedback.
-
|
|
Beta Was this translation helpful? Give feedback.
-
|
Dear villekf, I am so sorry. I was able to get a reconstructed image!! I have tried and failed with many software. Please let me ask you another question sometime. thank you |
Beta Was this translation helpful? Give feedback.
-
|
Next release will print out an error in such cases. |
Beta Was this translation helpful? Give feedback.
-
Hi,
The OMEGA framework is an impressive tool for PET reconstruction and a lot can be learned while going through individual m files.
Upon OMEGA installation (using Octave) and running install_mex I got messages ...."Implementations 1 & 4 built" and "Implementation 3 built". However, when testing the "lor_pixel_count_prepass.m" code an error: "'projector_oct' undefined" appears although the source folder contains .oct files. Any ideas why this happens?
Best regards,
Tom
Beta Was this translation helpful? Give feedback.
All reactions