Skip to content

Loops over tables in the job's database - #38

Merged
seamm merged 3 commits into
mainfrom
dev
Oct 3, 2026
Merged

Loops over tables in the job's database#38
seamm merged 3 commits into
mainfrom
dev

Conversation

@paulsaxe

@paulsaxe paulsaxe commented Oct 3, 2026

Copy link
Copy Markdown
Contributor
  • Loops over the rows of a table work with tables kept in the job's database (seamm 2026.10.3); the job's database is committed after each step of the loop body.
  • Bugfix: selecting rows with a test on a numeric column failed when the second value was empty, although only "between" uses it.
  • Rows can be selected on the table's index column too.
  • The values of _row and of the row variables are plain Python values.

🤖 Generated with Claude Code

paulsaxe and others added 3 commits October 3, 2026 11:14
@paulsaxe @claude
Loop over table rows with seamm.Table; commit after each body step
6ba6156 
Also fixes the row selection failing for numeric columns when the second value is
empty (it is only needed for between).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@paulsaxe @claude
Use seamm.step_completed after each body step
8327ae6 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@paulsaxe @claude
Release 2026.10.3: pin seamm>=2026.10.3; HISTORY; CI on uv (no test_e…
4fdb799 
…nv.yaml)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@paulsaxe paulsaxe added bug Something isn't working enhancement New feature or request labels Oct 3, 2026
@paulsaxe
paulsaxe requested a review from seamm October 3, 2026 18:25
seamm
seamm approved these changes Oct 3, 2026
View reviewed changes

@seamm seamm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@seamm
seamm merged commit 8d143d8 into main Oct 3, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@seamm seamm seamm approved these changes

Assignees

No one assigned

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@paulsaxe @seamm