Skip to content

Fix SyntaxError: duplicate update_header kwarg in RandomControlSourcePathReceiver - #23

Closed
bzwink wants to merge 1 commit into
sandialabs:mainfrom
bzwink:fix/spr-duplicate-kwarg
Closed

bzwink wants to merge 1 commit into
sandialabs:mainfrom
bzwink:fix/spr-duplicate-kwarg

Conversation

@bzwink

@bzwink bzwink commented Jul 10, 2026

Copy link
Copy Markdown

The auto_truncation_by_l_curve dispatch branch in RandomControlSourcePathReceiver.system_id_update passes update_header=False twice. Repeated keyword arguments are a SyntaxError at compile time (line 357), so the entire spr_random_control_law module fails to import — breaking every ISE technique, not just auto_truncation_by_l_curve. This removes the duplicate.

Found while running the SPR random control law as a Rattlesnake control law; verified the module compiles and the auto_tikhonov_by_l_curve path runs end-to-end in headless Rattlesnake simulations.

🤖 Generated with Claude Code

…PathReceiver

The auto_truncation_by_l_curve dispatch branch in
RandomControlSourcePathReceiver.system_id_update passed
update_header=False twice, which is a SyntaxError at parse time --
the entire spr_random_control_law module fails to import, breaking
every ISE technique (not just auto_truncation_by_l_curve).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@bzwink

bzwink commented Sep 23, 2026

Copy link
Copy Markdown
Author

Closing as superseded: the duplicate update_header keyword fix was merged to main in #25.

@bzwink bzwink closed this Sep 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant