Gemma Fardell

Results 50 comments of Gemma Fardell

I'm happy to have a look at implementing this.

I've had a quick look at this [function](https://github.com/vais-ral/CCPi-Regularisation-Toolkit/blob/master/src/Core/regularisers_CPU/ROF_TV_core.c#L270L316). A simple way without changing the code too much, and avoiding repetition would be to pass a pointer to lambda and a...

I think we want 'https://tomographicimaging.github.io/CIL' to be the CIL entry point. Currently that's probably the github CIL readme.md, but we also have a page on https://ccpi.ac.uk/cil/ . This would mean...

`test_SPDHG_vs_PDHG_explicit` and `test_SPDHG_vs_PDHG_implicit` are particularly slow. The `assert_almost_equal` on such a large array is probably one of the biggest costs.

I don't really like how things are implemented in 'label.py'. I can see what it's trying to achieve and agree something is needed but I think it needs some thought....

> - We need to add Josh and Nick to https://github.com/TomographicImaging/CIL/blob/master/NOTICE.txt. > - We also need to add them as authors to files where they added code (I guess use...

![image](https://github.com/user-attachments/assets/c4f4f552-d59a-46d3-bb61-e54afb1d1cd2) The documentation is a bit odd. The class name is strange, and the white space between the attributes. I guess it's useful to have the classes in the documentation,...

This is also needed for nikon data collected with IPC. I think as all the raw projections, dark and flat are all needed to describe the data they should be...

We agreed to pass 3 AcqData (data, flat and dark) to the normaliser, which will return a new AcqData. We also need to enhance the normalisation processor to work as:...

I'm surprised that conda doesn't set the path automatically when you activate it. I'm pretty sure that's the intended behaviour. I think adding it to "$CONDA_PREFIX$/etc/conda/activate.d" and "$CONDA_PREFIX$/etc/conda/deactivate.d" is the...