<div dir="ltr">Hello Alex, <div><br></div><div>I just merge your pull-request, thanks again for the contribution ! Please for the future, if possible try to propose several smaller "pull request" rather than a single big one, it makes it more simple for me to update.</div><div><br></div><div>regarding the question about the three classes and the same ReadTranslationTable, this is probably not intented but the history of Gate is close to the one of the dinosaurs, so I dont know ;) As you describe in the github thread (<a href="https://github.com/OpenGATE/Gate/issues/18">https://github.com/OpenGATE/Gate/issues/18</a>) there is probably a bug somewhere around material affectation in this configuration. To be sure I understand:</div><div><br></div><div>- case1: analytical phantom + gps = correct result</div><div>- case2: voxelized phantom + gps = correct result<br></div><div>- case3: analytical phantom + voxelized source = correct result<br></div><div>- case4: voxelized phantom + voxelized source = wrong result<br></div><div><br></div><div>is that true ? </div><div><br></div><div>David</div><div><br></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 5, 2015 at 10:33 PM, Alex Vergara Gil <span dir="ltr"><<a href="mailto:alexvergaragil@gmail.com" target="_blank">alexvergaragil@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Dear All<br>
<br>
I have created an Issue on github for this thread where I explain my<br>
problems (and my findings).<br>
<br>
Regards<br>
Alex<br>
<br>
2015-05-04 12:11 GMT-04:00, Alex Vergara Gil <<a href="mailto:alexvergaragil@gmail.com">alexvergaragil@gmail.com</a>>:<br>
<div class="HOEnZb"><div class="h5">> Dear All<br>
><br>
> I have seen three files<br>
><br>
> GateVImageVolume<br>
> GateGeometryVoxelRangeTranslator<br>
> GateGeometryVoxelTabulatedTranslator<br>
><br>
> with the same ReadTranslationTable procedure. Is this intended? Is<br>
> there an intention to put this procedure in the last two classes?<br>
> There is currently no relation between these three classes more than<br>
> the last two inherits from the same class. What is the intention in<br>
> this case? It affects how a translation table is read and passed to<br>
> G4.<br>
><br>
> Regards<br>
> Alex<br>
><br>
> 2015-04-30 11:26 GMT-04:00, Alex Vergara Gil <<a href="mailto:alexvergaragil@gmail.com">alexvergaragil@gmail.com</a>>:<br>
>> Dear all<br>
>><br>
>> I think I have found a posible bug related to my issue (possible wrong<br>
>> material handler in voxelized range translator, giving bad density<br>
>> values and therefore bad dose calculations). In the file<br>
>> GateGeometryVoxelRangeTranslator.cc, specifically in the function<br>
>><br>
>> G4String GateGeometryVoxelRangeTranslator::GetNextMaterial(G4bool<br>
>> doReset)<br>
>> {<br>
>>   static GateVoxelMaterialTranslationRangeVector::iterator anIterator<br>
>> = m_voxelMaterialTranslation.begin();<br>
>><br>
>>   if (doReset)<br>
>>     anIterator = m_voxelMaterialTranslation.begin();<br>
>><br>
>>   G4String aMaterial = ( anIterator!=m_voxelMaterialTranslation.end()<br>
>> ) ? anIterator->second : G4String("") ;<br>
>>   if (aMaterial!="")<br>
>>     anIterator++;<br>
>><br>
>>   return aMaterial;<br>
>> }<br>
>><br>
>> Here you may see that the iterator is initialized every time the<br>
>> function is called, I think it is better to put the iterator as a<br>
>> variable, but you know better than me, so please could you explain why<br>
>> it is used a static iterator here?<br>
>><br>
>> Regards<br>
>> Alex<br>
>><br>
>><br>
>> 2015-04-30 8:07 GMT-04:00, Kiran Joshi <<a href="mailto:kiran.j88@gmail.com">kiran.j88@gmail.com</a>>:<br>
>>> On Thu, 30 Apr 2015 at 13:01 Alex Vergara Gil <<a href="mailto:alexvergaragil@gmail.com">alexvergaragil@gmail.com</a>><br>
>>> wrote:<br>
>>><br>
>>>> Dear all<br>
>>>><br>
>>>> Attached I explain my problems.<br>
>>>><br>
>>>> Regards<br>
>>>> Alex<br>
>>>> _______________________________________________<br>
>>>> Gate-users mailing list<br>
>>>> <a href="mailto:Gate-users@lists.opengatecollaboration.org">Gate-users@lists.opengatecollaboration.org</a><br>
>>>> <a href="http://lists.opengatecollaboration.org/mailman/listinfo/gate-users" target="_blank">http://lists.opengatecollaboration.org/mailman/listinfo/gate-users</a><br>
>>><br>
>><br>
><br>
_______________________________________________<br>
Gate-users mailing list<br>
<a href="mailto:Gate-users@lists.opengatecollaboration.org">Gate-users@lists.opengatecollaboration.org</a><br>
<a href="http://lists.opengatecollaboration.org/mailman/listinfo/gate-users" target="_blank">http://lists.opengatecollaboration.org/mailman/listinfo/gate-users</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">David Sarrut, Phd<br>Directeur de recherche CNRS<br>CREATIS, UMR CNRS 5220, Inserm U 1044<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>