[Castor-users] Possible problem with module axial spacing modelization in the recon utility

[Maxime Toussaint]

Hello,

I think I have found where was the problem that created the error I talked about in my previous mail.

The line 644-645 of iScannerPET.cc are:
     FLTNBLUT size_trs_rsector = nb_trs_mod*size_trs_mod + (nb_trs_submod-1)*gap_trs_mod;
     FLTNBLUT size_axl_rsector = nb_axl_mod*size_axl_mod + (nb_axl_submod-1)*gap_axl_mod;
But it would make more sense if they were:
     FLTNBLUT size_trs_rsector = nb_trs_mod*size_trs_mod + (nb_trs_mod-1)*gap_trs_mod;
     FLTNBLUT size_axl_rsector = nb_axl_mod*size_axl_mod + (nb_axl_mod-1)*gap_axl_mod;

I recompiled Castor with that modification and the problem I observed previously are no longer there. I didn't make any further tests. However, I think it is pretty straightforward, thanks to the name of the variables.

By the way, is castor hosted somewhere like github and, in the affirmative, should I make a pull request?

Have a nice day,
Maxime Toussaint