Skip to content

delay tigre import#2323

Merged
casperdcl merged 2 commits into
masterfrom
delay-imports
May 8, 2026
Merged

delay tigre import#2323
casperdcl merged 2 commits into
masterfrom
delay-imports

Conversation

@casperdcl
Copy link
Copy Markdown
Member

@casperdcl casperdcl commented May 8, 2026

Probably related/required for:

Testing

Manually verified that the docs CI action:

  • does not install tigre
  • produces DocumentationHTML art[ie]fact which contains tigre plugin:
mkdir CIL; unzip -d CIL ~/Downloads/DocumentationHTML.zip
python -m http.server &
$BROWSER "http://localhost:8000/CIL/2323_merge/plugins/#tigre"
image

@casperdcl casperdcl added this to the v26.0 milestone May 8, 2026
@casperdcl casperdcl self-assigned this May 8, 2026
@casperdcl casperdcl requested a review from a team as a code owner May 8, 2026 11:44
@github-project-automation github-project-automation Bot moved this to Todo in CIL work May 8, 2026
@casperdcl casperdcl moved this from Todo to Priority review in CIL work May 8, 2026
@casperdcl casperdcl changed the title Delay imports delay tigre import May 8, 2026
@casperdcl casperdcl mentioned this pull request May 8, 2026
9 tasks
@casperdcl casperdcl removed request for gfardell and hrobarts May 8, 2026 12:06
Comment thread Wrappers/Python/cil/plugins/tigre/FBP.py Outdated
@lauramurgatroyd
Copy link
Copy Markdown
Member

Hi @casperdcl I had been working on this locally and my version was almost identical to what you've done 👍

@casperdcl casperdcl merged commit 5ecdd83 into master May 8, 2026
11 checks passed
@casperdcl casperdcl deleted the delay-imports branch May 8, 2026 13:35
@github-project-automation github-project-automation Bot moved this from Priority review to Done in CIL work May 8, 2026
This was referenced May 8, 2026
casperdcl added a commit that referenced this pull request May 8, 2026
casperdcl added a commit that referenced this pull request May 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move tigre imports to point of use, instead of at top of files using tigre

2 participants