Skip to content

Use ccpi channel instead of tomography mirror in actions#2311

Merged
lauramurgatroyd merged 2 commits intomasterfrom
use_ccpi_in_actions
Apr 17, 2026
Merged

Use ccpi channel instead of tomography mirror in actions#2311
lauramurgatroyd merged 2 commits intomasterfrom
use_ccpi_in_actions

Conversation

@lauramurgatroyd
Copy link
Copy Markdown
Member

@lauramurgatroyd lauramurgatroyd commented Apr 16, 2026

Description

Using the ccpi channel instead of the tomography mirror as we had problems with timeout errors - see #2308

The ccpi channel usage instead seems to resolve or minimise time out issues

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

Testing you performed

Please add any demo scripts to https://github.com/TomographicImaging/CIL-Demos/tree/main/misc
Ran all tests and it passed first time: https://github.com/TomographicImaging/CIL/actions/runs/24522518083

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

@lauramurgatroyd
Copy link
Copy Markdown
Member Author

lauramurgatroyd commented Apr 17, 2026

The first time the actions ran on this PR we had one failure - the docs action. This is not the action we have been having problems with in #2308
The issue was:
urllib.error.URLError: <urlopen error [Errno 110] Connection timed out>
make: *** [Makefile:32: dirhtml] Error 1
Error: Process completed with exit code 2.

No issues on rerunning.

When running all of the tests (like would be done if merged to master), everything passed first time! Which I have not been able to achieve on master or on any other branch that uses the tomography mirror instead of the ccpi channel. Therefore I suggest we merge this.

The all-tests run:
https://github.com/TomographicImaging/CIL/actions/runs/24522518083

@gfardell gfardell self-requested a review April 17, 2026 10:29
@lauramurgatroyd lauramurgatroyd marked this pull request as ready for review April 17, 2026 10:36
@lauramurgatroyd lauramurgatroyd requested a review from a team as a code owner April 17, 2026 10:36
@github-project-automation github-project-automation bot moved this to Todo in CIL work Apr 17, 2026
@lauramurgatroyd lauramurgatroyd moved this from Todo to Priority review in CIL work Apr 17, 2026
@lauramurgatroyd lauramurgatroyd merged commit fac074b into master Apr 17, 2026
51 of 52 checks passed
@github-project-automation github-project-automation bot moved this from Priority review to Done in CIL work Apr 17, 2026
@lauramurgatroyd lauramurgatroyd deleted the use_ccpi_in_actions branch April 17, 2026 10:54
@lauramurgatroyd lauramurgatroyd mentioned this pull request Apr 17, 2026
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants