Skip to content

Conversation

@fso42
Copy link
Contributor

@fso42 fso42 commented Nov 18, 2025

Summary

This PR adds command-line argument support to runCom6Scarp.py for overriding the scarp analysis
method configuration, enabling users to switch between plane and ellipsoid methods without modifying
configuration files.

Changes

  • Added --method / -m command-line argument to runCom6Scarp.py
    • Accepts values: plane, ellipsoid, or ini (default)
    • Validates input through argparse choices
  • Extended runScarpAnalysisWorkflow() function signature
    • Added method parameter for programmatic control
    • Updated docstring to document the new parameter
  • Implemented configuration override logic
    • Overrides scarpCfg["SETTINGS"]["method"] when --method plane or --method ellipsoid is specified
    • Falls back to configuration file default when --method ini or no argument provided
    • Logs when using ini file default for transparency

- Introduced `method` parameter to `runScarpAnalysisWorkflow` for selecting analysis method: `plane`, `ellipsoid`, or `ini`.
- Updated command-line interface to accept `--method` argument to override the default configuration.
@fso42 fso42 self-assigned this Nov 18, 2025
@fso42 fso42 added the enhancement New feature or request label Nov 18, 2025
@qltysh
Copy link
Contributor

qltysh bot commented Nov 18, 2025

❌ 1 blocking issue (1 total)

Tool Category Rule Count
ruff Style Incorrect formatting, autoformat by running qlty fmt. 1

@qltysh one-click actions:

  • Auto-fix formatting (qlty fmt && git push)

- Reformatted long `assert` statements with parentheses for better clarity.
@fso42 fso42 merged commit 8012dd4 into master Nov 18, 2025
4 of 5 checks passed
@fso42 fso42 deleted the addMethodScarp branch November 18, 2025 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants