refactor: move terratorch_iterate under src/claimed/iterate#326
Merged
Conversation
Relocate the bundled iterate sources from the top-level terratorch_iterate/ package into src/claimed/iterate/, so all first-party packages live under a single src/ root and the iterate code is namespaced under claimed. - git mv terratorch_iterate -> src/claimed/iterate - rewrite the import path terratorch_iterate -> claimed.iterate across all Python sources and tests, and in the coordinator plugin loader's built-in module list - console scripts: iterate / iterate-classic now point at claimed.iterate.iterate2:main / claimed.iterate.main:main - packages.find: drop the obsolete top-level terratorch_iterate* include; claimed.iterate is picked up by the existing claimed* rule under src/ - CI: flake8/pylint/docs path globs no longer reference terratorch_iterate/ - docs: mkdocstrings ':::terratorch_iterate.*' references -> ':::claimed.iterate.*' No behavioral change. Verified: all iterate submodules import, coordinator plugins load, iterate/iterate-classic entry points run, and unit + coordinator integration tests pass (test_iterate2 shell-out included). Signed-off-by: Romeo Kienzler <romeo.kienzler1@ibm.com>
pull Bot
pushed a commit
to zongruxie4/component-library
that referenced
this pull request
Jul 7, 2026
Release includes the database-backed job coordinator (claimed-framework#325) and the relocation of the iterate sources to claimed.iterate (claimed-framework#326). Signed-off-by: Romeo Kienzler <romeo.kienzler1@ibm.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Relocates the bundled iterate sources from the top-level
terratorch_iterate/package intosrc/claimed/iterate/, so all first-party packages live under a singlesrc/root and the iterate code is namespaced underclaimed.Pure move + rename — no behavioral change.
Changes
git mv terratorch_iterate → src/claimed/iterate(tracked as renames)terratorch_iterate→claimed.iterateacross all Python sources and tests, and in the coordinator plugin loader's built-in module listiterate/iterate-classicnow point atclaimed.iterate.iterate2:main/claimed.iterate.main:mainpackages.find: drop the obsolete top-levelterratorch_iterate*include;claimed.iterateis picked up by the existingclaimed*rule undersrc/flake8/pylint/docs-checkpath globs no longer referenceterratorch_iterate/:::terratorch_iterate.*references →:::claimed.iterate.*Verification
claimed.iterate.*; setuptools discovers all 6 subpackagesiterate/iterate-classicentry points regenerate and runtests/unit+ coordinator integration tests, includingtest_iterate2(which shells out to theiteratecommand)Note: the only remaining
terratorch_iteratestring in the tree is a filesystem path (/u/ltizzei/test_terratorch_iterate) inside a test config oracle — intentionally left as-is.