Skip to content

Standard structure selection for loops over systems - #37

Merged
seamm merged 3 commits into
mainfrom
dev
Sep 18, 2026
Merged

Standard structure selection for loops over systems#37
seamm merged 3 commits into
mainfrom
dev

Conversation

@paulsaxe

@paulsaxe paulsaxe commented Sep 18, 2026

Copy link
Copy Markdown
Contributor
  • The "For systems in the database" loop now uses SEAMM's standard structure selection: the systems to loop over (all, the current one, or those whose name is / matches / matches a regular expression) and which configurations of each (last, first, all, the current one, or by name), or a variable holding a list of configurations or systems. The defaults -- every system, its last configuration -- are unchanged, and flowcharts saved with the old options are translated when loaded.
  • Bugfix: selecting configurations by "name is" never matched (the name was compared by identity, not equality), and "name matches" / "name regexp" fell through to no selection at all.
  • Requires seamm 2026.9.18.1 or later.

🤖 Generated with Claude Code

paulsaxe and others added 3 commits November 22, 2025 14:22
@paulsaxe
Fixed wrong month on release date
0c44c6d
@paulsaxe @claude
Add 'update' make target for post-release main/dev sync
58007c3 
Self-documented target that checks out and pulls main, fast-forwards
dev onto it (git merge --ff-only as a guardrail), runs lint/install/test,
then pushes dev -- codifying the manual post-release workflow.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@paulsaxe @claude
Standard structure selection for loops over systems
eda4ba0 
The "For systems in the database" loop uses seamm's standard structure
selection (defaults unchanged: every system, its last configuration), with the
old option names and values translated when an older flowchart is loaded. Fixes
"name is" never matching (identity comparison) and "name matches" / "name
regexp" falling through to no selection. CI environment moved to the pip-based
layout.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@paulsaxe paulsaxe added bug Something isn't working enhancement New feature or request labels Sep 18, 2026
@paulsaxe
paulsaxe requested a review from seamm September 18, 2026 16:55
seamm
seamm approved these changes Sep 18, 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 a20c30a into main Sep 18, 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