Edoardo Pasca

Results 183 comments of Edoardo Pasca

I'll provide an example. ATM I am using it in the electron tomography data.

As I said, it is for gradient on all dimensions. Since it works only with the C backend of the `Gradient`, to be able to use it you need to...

I'm doing some more basic tests and I found out that the norm we calculate normally with `np.dot` differs quite a lot with the one calculated with `Gradient_C`. For instance...

It is clear we cannot accept a PR without unittests

We have developed something along these lines in the CILViewer with the [`cilBaseResampleReader`](https://github.com/vais-ral/CILViewer/blob/master/Wrappers/Python/ccpi/viewer/utils/conversion.py#L568) and subclasses. These are different from the normal `Reader`s that would return you the whole dataset. The...

@jakobsj we have a default for angle unit. The angle unit is not written in the Zeiss file. So, even if it's not needed it's nice to specify it.

It looks to me that everyone is actually making sure that the number of subsets are divisors of the number of projections. It doesn't quite make sense to me, though.

I believe that at least [boost is built in source](https://github.com/CCPPETMR/SIRF-SuperBuild/blob/master/SuperBuild/External_Boost.cmake#L60). Do you really want to error out?

Ah, I had totally misunderstood that point. Then I agree.