Update CIL build instructions in developer guide by paskino · Pull Request #2286 · TomographicImaging/CIL

paskino · 2026-02-02T13:53:20Z

Description

Updates instructions on local build of CIL.
- README
- Developer's guide
Updates to FindIPP.cmake to find IPP libraries on Windows

Example Usage

Contribution Notes

The content of this Pull Request (the Contribution) is intentionally submitted for inclusion in CIL (the Work) under the terms and conditions of the Apache-2.0 License
I confirm that the contribution does not violate any intellectual property rights of third parties

Changes

README.md
Updates the build script as locally I noticed that IPP was not found unless -Ccmake.define.IPP_ROOT=$CONDA_PREFIX was passed
Created environment files:
- Development envs scripts#9

Testing you performed

Please add any demo scripts to https://github.com/TomographicImaging/CIL-Demos/tree/main/misc

Local build on:

Linux
mac ARM
Windows

Checked the artifact documentation produced by GHA.

Linux

On Linux a total of 43 tests are skipped:

test_aqdata_full_tigre (test_DataProcessor.TestBinner.test_aqdata_full_tigre) 
test_pad_id_full_tigre (test_DataProcessor.TestPadder.test_pad_id_full_tigre) 
test_aqdata_full_tigre (test_DataProcessor.TestSlicer.test_aqdata_full_tigre) 
test_linearity (test_PluginsTigre_Siddon.Test_Parallel2D_Projectors_toy_tigre.test_linearity) TIGRE backprojector weights bug
test_norm (test_PluginsTigre_Siddon.Test_Parallel2D_Projectors_toy_tigre.test_norm) TIGRE backprojector weights bug
test_linearity (test_PluginsTigre_Siddon.Test_Parallel3D_Projectors_toy_tigre.test_linearity) TIGRE backprojector weights bug
test_norm (test_PluginsTigre_Siddon.Test_Parallel3D_Projectors_toy_tigre.test_norm) TIGRE backprojector weights bug
test_linearity (test_PluginsTigre_interpolated.Test_Parallel2D_Projectors_toy_tigre.test_linearity) TIGRE backprojector weights bug
test_norm (test_PluginsTigre_interpolated.Test_Parallel2D_Projectors_toy_tigre.test_norm) TIGRE backprojector weights bug
test_linearity (test_PluginsTigre_interpolated.Test_Parallel3D_Projectors_toy_tigre.test_linearity) TIGRE backprojector weights bug
test_norm (test_PluginsTigre_interpolated.Test_Parallel3D_Projectors_toy_tigre.test_norm) TIGRE backprojector weights bug
test_geometry_gpu (test_io.TestZeissDataReader.test_geometry_gpu) Missing prerequisites: has_file False, has_olefile True has_dxchange True
test_import_error (test_io.TestZeissDataReader.test_import_error) This unit test runs only if dxchange is not installed: has_dxchange True
test_read_and_reconstruct_2D_gpu (test_io.TestZeissDataReader.test_read_and_reconstruct_2D_gpu) Missing prerequisites: has_file False, has_olefile True has_dxchange True, has_astra True has_wget True
test_read_txm_recon_file_gpu (test_io.TestZeissDataReader.test_read_txm_recon_file_gpu) Missing prerequisites: has_file False, has_recon_file False has_olefile True has_dxchange True, has_astra True has_wget True

Plus these which test compatibility with SIRF (which is not installed)

test_TNV_call_works (test_SIRF.TestPETRegularisation.test_TNV_call_works) TNV not implemented for 2D
test_TNV_proximal_works (test_SIRF.TestPETRegularisation.test_TNV_proximal_works) TNV not implemented for 2D
test_SIRF_CIL_MLEM (test_SIRF.TestCILSIRFPrecond.test_SIRF_CIL_MLEM) Skipping as SIRF is not available
test_Gradient (test_SIRF.TestGradientMR_2D.test_Gradient) Skipping as SIRF is not available
test_TVdenoisingMR (test_SIRF.TestGradientMR_2D.test_TVdenoisingMR) Has SIRF
test_Gradient (test_SIRF.TestGradientPET_2D.test_Gradient) Skipping as SIRF is not available
test_Gradient (test_SIRF.TestGradientPET_3D.test_Gradient) Skipping as SIRF is not available
test_FGP_TV_call_works (test_SIRF.TestMRRegularisation.test_FGP_TV_call_works) Has SIRF and CCPi Regularisation
test_FGP_TV_proximal_works (test_SIRF.TestMRRegularisation.test_FGP_TV_proximal_works) Has SIRF and CCPi Regularisation
test_TGV_call_works (test_SIRF.TestMRRegularisation.test_TGV_call_works) Has SIRF and CCPi Regularisation
test_TGV_proximal_works (test_SIRF.TestMRRegularisation.test_TGV_proximal_works) Has SIRF and CCPi Regularisation
test_TNV_call_works (test_SIRF.TestMRRegularisation.test_TNV_call_works) Has SIRF and CCPi Regularisation
test_TNV_proximal_works (test_SIRF.TestMRRegularisation.test_TNV_proximal_works) Has SIRF and CCPi Regularisation
test_FGP_TV_call_works (test_SIRF.TestPETRegularisation.test_FGP_TV_call_works) Has SIRF and CCPi Regularisation
test_FGP_TV_proximal_works (test_SIRF.TestPETRegularisation.test_FGP_TV_proximal_works) Has SIRF and CCPi Regularisation
test_TGV_call_works (test_SIRF.TestPETRegularisation.test_TGV_call_works) Has SIRF and CCPi Regularisation
test_TGV_proximal_works (test_SIRF.TestPETRegularisation.test_TGV_proximal_works) Has SIRF and CCPi Regularisation
test_FGP_TV_call_works (test_SIRF.TestRegRegularisation.test_FGP_TV_call_works) Has SIRF and CCPi Regularisation
test_FGP_TV_proximal_works (test_SIRF.TestRegRegularisation.test_FGP_TV_proximal_works) Has SIRF and CCPi Regularisation
test_TGV_call_works (test_SIRF.TestRegRegularisation.test_TGV_call_works) Has SIRF and CCPi Regularisation
test_TGV_proximal_works (test_SIRF.TestRegRegularisation.test_TGV_proximal_works) Has SIRF and CCPi Regularisation
test_TNV_call_works (test_SIRF.TestRegRegularisation.test_TNV_call_works) Has SIRF and CCPi Regularisation
test_TNV_proximal_works (test_SIRF.TestRegRegularisation.test_TNV_proximal_works) Has SIRF and CCPi Regularisation
test_BlockDataContainer_with_SIRF_DataContainer_add (test_SIRF.TestSIRFCILIntegration.test_BlockDataContainer_with_SIRF_DataContainer_add) Has SIRF
test_BlockDataContainer_with_SIRF_DataContainer_divide (test_SIRF.TestSIRFCILIntegration.test_BlockDataContainer_with_SIRF_DataContainer_divide) Has SIRF
test_BlockDataContainer_with_SIRF_DataContainer_multiply (test_SIRF.TestSIRFCILIntegration.test_BlockDataContainer_with_SIRF_DataContainer_multiply) Has SIRF
test_BlockDataContainer_with_SIRF_DataContainer_subtract (test_SIRF.TestSIRFCILIntegration.test_BlockDataContainer_with_SIRF_DataContainer_subtract) Has SIRF

Windows

======================================================================
FAIL: test_set_up (test_algorithms.TestLSQR.test_set_up)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\Users\ofn77899\Dev\CIL\Wrappers\Python\test\test_algorithms.py", line 2059, in test_set_up
    self.assertAlmostEqual(lsqr.beta,  beta, 5 )
AssertionError: 96.19578 != 96.19578622941998 within 5 places (8.33635357366802e-06 difference)

----------------------------------------------------------------------
Ran 1125 tests in 226.381s

FAILED (failures=1, skipped=43)

Related issues/links

Checklist

I have performed a self-review of my code
I have added docstrings in line with the guidance in the developer guide
I have updated the relevant documentation
I have implemented unit tests that cover any new or modified functionality
CHANGELOG.md has been updated with any functionality change
Request review from all relevant developers

data is not installed by the pip command so it is not useful to clone the repo recursively

Added information about creating a shallow clone with --depth parameter.

casperdcl

a bit concerned about:

repetition in README.md & developer_guide.rst (would prefer one to link to the other)
lack of mention of conda env names (create --name <env_name> && activate <env_name)

Co-authored-by: Casper da Costa-Luis <casper.dcl@physics.org> Co-authored-by: Laura Murgatroyd <60604372+lauramurgatroyd@users.noreply.github.com> Co-authored-by: Edoardo Pasca <14138589+paskino@users.noreply.github.com> Signed-off-by: Laura Murgatroyd <60604372+lauramurgatroyd@users.noreply.github.com>

lauramurgatroyd · 2026-04-14T11:10:14Z

Some screenshots of the rendered developer guide:

lauramurgatroyd · 2026-04-14T11:11:17Z

Hi, this PR is now ready for review again. I actioned the changes I suggested. Please see the unresolved threads for open questions.

gfardell

I'm confused by the environment file links not working, maybe they are in an unmerged PR?

However, I would have thought that using the environment files that ship with CIL is good enough as users will need to clone the repositories by default. Was there a reason to move away from that?

Co-authored-by: Gemma Fardell <47746591+gfardell@users.noreply.github.com> Signed-off-by: Laura Murgatroyd <60604372+lauramurgatroyd@users.noreply.github.com>

lauramurgatroyd · 2026-04-16T14:53:21Z

I'm confused by the environment file links not working, maybe they are in an unmerged PR?

However, I would have thought that using the environment files that ship with CIL is good enough as users will need to clone the repositories by default. Was there a reason to move away from that?

This PR in scripts has the env file: https://github.com/TomographicImaging/scripts/pull/9/changes

I am not sure where they should live - we need to discuss what the purpose of https://github.com/TomographicImaging/scripts is

Signed-off-by: Casper da Costa-Luis <casper.dcl@physics.org>

casperdcl

We could also move scripts/requirements-test.yml to just environment.yml in the root.

lauramurgatroyd · 2026-04-17T16:37:36Z

We will move the dev environment files into CIL, not host them in scripts, see: TomographicImaging/scripts#9 (review)

Co-authored-by: Gemma Fardell <47746591+gfardell@users.noreply.github.com> Signed-off-by: Laura Murgatroyd <60604372+lauramurgatroyd@users.noreply.github.com>

lauramurgatroyd · 2026-04-24T10:35:18Z

Some of the new docs changes rendered:

paskino and others added 6 commits February 6, 2023 15:03

rempove py36 np1.15, GHA to test py39 np122

998c25a

update readme

250cb03

updated clone and build command

42af679

data is not installed by the pip command so it is not useful to clone the repo recursively

update pip command to find IPP

3c1b642

fix recipe to find IPP

482a640

Merge branch 'master' into update_dev_instructions

d227e3c

paskino added this to UM 2026 Feb 2, 2026

github-project-automation Bot moved this to Todo in UM 2026 Feb 2, 2026

paskino moved this from Todo to In Progress in UM 2026 Feb 2, 2026

update pip command

d37a157

paskino mentioned this pull request Feb 5, 2026

ctests fail on Windows due to module problem #2141

Closed

paskino added 3 commits February 6, 2026 09:35

add search CMake search path for WIN32 IPP

758f315

added instructions on developer guide

6a098d5

updated changelog

e597d53

paskino marked this pull request as ready for review February 6, 2026 20:43

paskino requested a review from a team as a code owner February 6, 2026 20:43

paskino requested review from casperdcl, gfardell and lauramurgatroyd and removed request for a team February 6, 2026 20:44

Update README with shallow clone instructions

8b3f51b

Added information about creating a shallow clone with --depth parameter.

paskino removed this from UM 2026 Feb 10, 2026

paskino added this to CIL work Feb 10, 2026

github-project-automation Bot moved this to Todo in CIL work Feb 10, 2026

gfardell requested review from Neonbluestoplight and hrobarts February 10, 2026 14:08

lauramurgatroyd reviewed Feb 10, 2026

View reviewed changes

Comment thread docs/source/developer_guide.rst Outdated

lauramurgatroyd reviewed Feb 10, 2026

View reviewed changes

Comment thread docs/source/developer_guide.rst Outdated

casperdcl requested changes Feb 17, 2026

View reviewed changes

lauramurgatroyd reviewed Feb 19, 2026

View reviewed changes

Comment thread README.md Outdated

lauramurgatroyd and others added 7 commits April 10, 2026 12:02

Merge branch 'master' into update_dev_instructions

52e7808

Move instructions out of readme

7e84ee4

update doc instructions and remove old scripts

222e9e6

Fix formatting

b8e7ed2

Rearrange docs

f56203b

Fix titles

9010a29

lauramurgatroyd moved this from Blocked to Priority review in CIL work Apr 15, 2026

lauramurgatroyd removed the request for review from hrobarts April 15, 2026 10:07

Merge branch 'master' into update_dev_instructions

79d14b4

gfardell reviewed Apr 16, 2026

View reviewed changes

Comment thread docs/source/developer_guide.rst

Apply suggestions from code review

66f0461

Co-authored-by: Gemma Fardell <47746591+gfardell@users.noreply.github.com> Signed-off-by: Laura Murgatroyd <60604372+lauramurgatroyd@users.noreply.github.com>

fix syntax

a0db0ab

Signed-off-by: Casper da Costa-Luis <casper.dcl@physics.org>

casperdcl approved these changes Apr 17, 2026

View reviewed changes

lauramurgatroyd and others added 4 commits April 17, 2026 17:39

Apply suggestions from code review

5b8eeaf

Co-authored-by: Gemma Fardell <47746591+gfardell@users.noreply.github.com> Signed-off-by: Laura Murgatroyd <60604372+lauramurgatroyd@users.noreply.github.com>

Merge remote-tracking branch 'upstream/master'

b92a0b1

merge master

5a29f83

Move environment files into this PR

b3ae694

lauramurgatroyd moved this from Priority review to In Progress in CIL work Apr 24, 2026

lauramurgatroyd added 2 commits April 24, 2026 11:20

Move env.yml scripts to live in CIL

669eaf0

tomophantom version

ebff6f1

lauramurgatroyd changed the title ~~Update dev instructions~~ Update CIL build instructions in developer guide Apr 24, 2026

update env instructions in docs instructions

0285624

lauramurgatroyd requested a review from gfardell April 24, 2026 10:40

Conversation

paskino commented Feb 2, 2026 • edited by lauramurgatroyd Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Example Usage

Contribution Notes

Changes

Testing you performed

Linux

Windows

Related issues/links

Checklist

Uh oh!

Uh oh!

Uh oh!

casperdcl left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

lauramurgatroyd commented Apr 14, 2026

Uh oh!

lauramurgatroyd commented Apr 14, 2026

Uh oh!

gfardell left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

lauramurgatroyd commented Apr 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

casperdcl left a comment

Choose a reason for hiding this comment

Uh oh!

lauramurgatroyd commented Apr 17, 2026

Uh oh!

lauramurgatroyd commented Apr 24, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

paskino commented Feb 2, 2026 •

edited by lauramurgatroyd

Loading

lauramurgatroyd commented Apr 16, 2026 •

edited

Loading