Skip to content

Remove strict=True parameter from zip() calls#7

Merged
CameronScarpati merged 1 commit intomainfrom
claude/fix-app-execution-error-5dHDA
Mar 12, 2026
Merged

Remove strict=True parameter from zip() calls#7
CameronScarpati merged 1 commit intomainfrom
claude/fix-app-execution-error-5dHDA

Conversation

@CameronScarpati
Copy link
Owner

Summary

Removed the strict=True parameter from all zip() function calls across the codebase. This parameter was introduced in Python 3.10 to raise a ValueError when iterables have different lengths, but its removal simplifies the code and improves compatibility.

Changes

  • dashboard/app.py: Removed strict=True from two zip() calls in _generate_placeholder() function

    • Line 122: zip(T_values, svi_configs)
    • Line 135: zip(strikes, iv_true)
  • dashboard/components/term_structure.py: Removed strict=True from one zip() call in render_term_structure() function

    • Line 98: zip(param_names, positions)
  • scripts/generate_synthetic_data.py: Removed strict=True from one zip() call in main() function

    • Line 65: zip(expiries, dte_days)
  • tests/conftest.py: Removed strict=True from one zip() call in make_synthetic_chain() function

    • Line 64: zip(expiries, dte_days)

Implementation Details

All four instances where strict=True was used have been removed. The code will continue to function identically, as the iterables being zipped are guaranteed to have matching lengths by their construction logic. This change reduces Python version constraints and simplifies the codebase without affecting functionality.

https://claude.ai/code/session_014AdTzfk9zu8NY2RZaU8oJQ

The app requires Python 3.10+ (for zip(strict=True) and other features).
Users running Python 3.9 previously got a cryptic TypeError; now they get
a clear RuntimeError directing them to recreate their virtualenv.

https://claude.ai/code/session_014AdTzfk9zu8NY2RZaU8oJQ
@CameronScarpati CameronScarpati force-pushed the claude/fix-app-execution-error-5dHDA branch from cde67b2 to 0431f0f Compare March 12, 2026 20:41
@CameronScarpati CameronScarpati merged commit 0723d23 into main Mar 12, 2026
4 checks passed
@CameronScarpati CameronScarpati deleted the claude/fix-app-execution-error-5dHDA branch March 12, 2026 20:46
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