<div dir="ltr"><div dir="ltr">Hi Zhengzhi, <br></div><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Feb 27, 2020 at 9:01 PM Zhengzhi Liu <<a href="mailto:zliu36@stanford.edu" target="_blank">zliu36@stanford.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Dear David, <div><br></div><div>Can I share some problems I have with Gate 8, which may be caused by bugs?  I believe I am not the only GATE user who came across these issues. </div></div></blockquote><div><br></div><div>sure ;) ! (If bug is confirmed, an issue should be filled on github)<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><br></div><div>Problem 1: <b> Coincident sorter</b></div><div>On top of the imcompatibility with genericRepeater as reported by Han Gyu, the built in coincident sorter is problematic in realistic situations. It works fine under ideal conditions as described in the Gate user manual v8.0. Taking the <b>cylindricalPET</b> as example, a cylindricalPET system can have 5 hierarchies at most, i.e., sector, module, submode, crystal, layer. This is to build an simplistic PET system. However, in real life, the crystal may have a wrapper, and detectors may be placed in some supporting structure and shielding. When I insert any other geometry into the system (for example, I added an aluminum wrapper for crystals, whether I attach it the system or not ), the coincidence sorter would fail immediately, but singles generation works fine. </div></div></blockquote><div><br></div><div>a first workaround has been proposed recently by Christian Pommaranz, see <a href="https://github.com/OpenGATE/Gate/pull/297" target="_blank">https://github.com/OpenGATE/Gate/pull/297</a> and <a href="https://github.com/OpenGATE/Gate/issues/296" target="_blank">https://github.com/OpenGATE/Gate/issues/296</a>. Could you try if it helps your issues? Please, note that the coincidence sorter does not work for combinations of angularRepeater and genericRepeater for now.</div><div><br></div><div>Please let us know if it helps ...<br></div><div><br></div><div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div></div><div>Problem 2: <b>work with voxelized source and Phantom</b></div><div>First, I can't get the built in example code (benchImageNavigators/pet) work. Second, it looks like no body has ever succeeded in visualing their voxelized source. The <a href="http://lists.opengatecollaboration.org/pipermail/gate-users/2010-April/004126.html" target="_blank">earliest email</a> about voxelized source visualization can date back to <b>10 years</b> agao, could you make an official reply to that or update it in the upcoming version? </div></div></blockquote><div><br></div><div>Please, could you please describe a bit more the first error? <br></div><div><br></div><div>Second, for the visualisation of the source, as proposed by Christopher and Matthew you may add an image at the exact same place. <br></div><br><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div></div><div>Thank you very much for your time reading and considering the problems.</div></div></blockquote><div><br></div><div>thanks to you for reporting ! <br></div><div><br></div><div>David</div><div>PS: I will be travelling this week, back on track next week ... <br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Sincerely </div><div><br></div><div>Zhengzhi </div><div><br></div><div></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Feb 19, 2020 at 7:21 AM David Sarrut <<a href="mailto:David.Sarrut@creatis.insa-lyon.fr" target="_blank">David.Sarrut@creatis.insa-lyon.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><br></div><div>excellent, thanks to both of you! <br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Feb 19, 2020 at 12:23 PM 강한규 <<a href="mailto:lovehangulp@naver.com" target="_blank">lovehangulp@naver.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div style="font-size:10pt;font-family:Gulim,sans-serif"><p>Dear David, and Christian,</p><p> Thanks for your kind response.</p><p> For the bug#1: I uploaded an issue about the GATE bug with cubicArray.(issue#296)</p><p> For the bug#2: Christian (included in this email) will handle the coincidence sorter issue wit his own pull requests (PR) on the git hub.</p><p> Best regards,</p><p> Han Gyu <br></p><p>--------</p><p>Han Gyu Kang, Ph.D, Researcher<br><br>National Institute of Radiological Sciences (NIRS)<br>National Institutes for Quantum and Radiological Science and Technology (QST)<br>  4-9-1 Anagawa, Inage-ku, Chiba 263-8555, JAPAN</p><p> </p><p style="font-size:10pt;font-family:sans-serif;padding:0px 0px 0px 10pt"><span>-----Original Message-----</span><br><b>From:</b> "David Sarrut"<<a href="mailto:David.Sarrut@creatis.insa-lyon.fr" target="_blank">David.Sarrut@creatis.insa-lyon.fr</a>> <br><b>To:</b> "강한규"<<a href="mailto:lovehangulp@naver.com" target="_blank">lovehangulp@naver.com</a>>; <br><b>Cc:</b> "gate-users"<<a href="mailto:gate-users@lists.opengatecollaboration.org" target="_blank">gate-users@lists.opengatecollaboration.org</a>>; <br><b>Sent:</b> 2020-02-19 (수) 17:25:22 (GMT+09:00)<br><b>Subject:</b> Re: [Gate-users] Towards Gate 9.0 (Report of GATEv8.2 bugs)<br> </p><div dir="ltr"><div>Hello 강한규, </div><div> </div><div>thanks very much for the very clear report! </div><div> </div><div>For bug#1: I am not expert in cubicArray repeater (I wrote the genericRepeater long time ago). We should investigate, but not sure it will be done for 9.0. Can you please put your email as an issue on github so that we keep memory of it? </div><div> </div><div>For bug#2: sorry, I cannot remember of the patch you mention (I probably forget this, my bad). Could you please provide the patch as a PR ? It will be included rapidly! </div><div> </div><div>Thanks a lot, do not hesitate to ask if need help for the PR</div><div> </div><div>Thank again, </div><div>David</div><div> </div><div> </div></div><div><div dir="ltr">On Wed, Feb 19, 2020 at 9:08 AM 강한규 <<a href="mailto:lovehangulp@naver.com" rel="noreferrer noopener" target="_blank">lovehangulp@naver.com</a>> wrote: </div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div style="font-size:10pt;font-family:Gulim,sans-serif"><p>Dear David Sarrut,</p><p> </p><p>Let me share the bugs of GATEv8.2.</p><p>I tried to simulate a <b>PEM (Positron emission mammography)</b> scanner with GATEv8.2.(slide#3-8)</p><p>For the system type, "<b>PETscanne</b>r" was used because of the non-fixed geometry.</p><p> </p><p><b>Bug#1: Check pattern with the cubicArray repeater.</b></p><p>When the back-to-back source (red) is at the center of the scanner, there was no problem.</p><p>However, as the source was placed close to the one detector side, the check pattern was appeared on the detector response.(slide#10)</p><p>This problem can be solved by using genericRepeater.(slide#11-12)</p><p>Unfortunately, however, the coincidence sorter does not work with the genericRepeater.</p><p> </p><p> </p><p><b>Bug#2: Coincidence sorter does not work with the genericRepeater (slide#13-16)</b></p><p>         The coincidence sorter issue could be solved by using git patch which was proposed by Christian Pommaranz (<a href="mailto:pommranz@astro.uni-tuebingen.de" rel="noreferrer noopener" target="_blank">pommranz@astro.uni-tuebingen.de</a>).</p><p><span style="font-size:10pt">         After several steps of GATE macro and source code modification, the PEM scanner simulation worked successfully.(slide#16)</span> </p><p style="font-size:13.3333px">         However, for the beginners of GATE, it is going to hard to do it by themselves.</p><div> Hopefully, the abovementioned bugs would be resolved in the GATEv9.0.</div><p> Best regards,</p><p> </p><p>Han Gyu Kang, Ph.D.</p><p> </p><p>Imaging Physics Group, NIRS-QST, Japan.</p><p> </p><p style="font-size:10pt;font-family:sans-serif;padding:0px 0px 0px 10pt"><span>-----Original Message-----</span><br><b>From:</b> "David Sarrut"<<a href="mailto:David.Sarrut@creatis.insa-lyon.fr" rel="noreferrer noopener" target="_blank">David.Sarrut@creatis.insa-lyon.fr</a>> <br><b>To:</b> "gate-users"<<a href="mailto:gate-users@lists.opengatecollaboration.org" rel="noreferrer noopener" target="_blank">gate-users@lists.opengatecollaboration.org</a>>; <br><b>Cc:</b> <br><b>Sent:</b> 2020-02-19 (수) 16:28:07 (GMT+09:00)<br><b>Subject:</b> [Gate-users] Towards Gate 9.0<br> </p><div dir="ltr">Dear all, <br><br>We are currently preparing the next Gate release, version 9.0, based on the current Geant4 release 10.6. This version includes a bunch of novelties compared to the previous 8.2 version (hence the jump in version number).<br><br>Please, if you want to include new feature or bug correction in this version, <u>submit your PullRequest (PR) asap</u>. We usually ask to have one PR with source code only in the main Gate repository, another PR with one example in the GateContrib repository, and some documentation changes via <a href="https://opengate.readthedocs.io/en/latest/" rel="noreferrer noopener" target="_blank">readthedocs</a>. In any case, do not hesitate to contact us for any help, we are happy to help !<br><br><b><u>For all</u></b>: please we need help to perform tests. Gate is a large beast and we still do not have a proper unit test system (working on it). This is simple: download and compile the branch "develop" and test it with your own simulation to see if anything is fine. Please, report us is anything seems strange. On our side, we have some benchmarks but they do not cover all the parts of Gate. <a href="https://en.wikipedia.org/wiki/Continuous_integration" rel="noreferrer noopener" target="_blank">Continuous integration</a> on github via Azure is currently being implemented (thanks to Thomas!) and will be ready soon.<div> </div><div>In advance, many many thanks to all recent contributors: Fatima, Anders, Mathieu, Thomas, David, Marta, Loic, Sam, Christian, Mateuz etc ... I am sure I forget people, sorry ... I will properly update the list of contributors in the following weeks :)<br><br>The ETA for the next release is early March. <div> </div><div>David</div></div></div></div></div></blockquote></div><div><br></div></div></div></blockquote></div></blockquote></div></blockquote></div><br clear="all"><br>-- <br><div dir="ltr"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr">David Sarrut, Phd<br>Directeur de recherche CNRS<br>CREATIS, UMR CNRS 5220, Inserm U1206<div>Centre de lutte contre le cancer Léon Bérard<br>28 rue Laënnec, 69373 Lyon cedex 08<br>Tel : 04 78 78 51 51 / 06 74 72 05 42<br><a href="http://www.creatis.insa-lyon.fr/~dsarrut" target="_blank">http://www.creatis.insa-lyon.fr/~dsarrut</a><br>_________________________________</div><div> "2 + 2 = 5,  for extremely large values of 2"<br>_________________________________</div></div></div></div></div></div></div></div>