Simplify suews-convert interface and fix remaining YAML conversion issues (builds on #581)#584
Merged
sunt05 merged 6 commits intofix/566-to-yaml-modulefrom Aug 6, 2025
Conversation
- Remove redundant 'yaml' option since 2025+ only supports YAML - Automatically determine conversion type based on target version: - Pre-2025: table-to-table conversion - 2025+: convert to YAML format - Fix all conversion issues preventing supy-to-yaml from working: - Handle empty DataFrame when STEBBS disabled - Add graceful handling of missing columns in legacy format - Fix water surface soilstore validation (allow 0 for water) - Provide defaults for missing config/description columns - Update all documentation to reflect simplified interface - Update tests to match new command structure Fixes #581
|
🤖 I've automatically formatted the code in this PR using:
Please pull the latest changes before making further edits. |
Co-authored-by: sunt05 <1802656+sunt05@users.noreply.github.com>
- Remove duplicate suews_convert_guide.rst - Update existing input_converter.rst with YAML conversion info - Consolidate all converter documentation in one place
- Replace deprecated delim_whitespace=True with sep=r'\s+' - Replace invalid quotechar=' ' with quoting=3 (QUOTE_NONE) - Fixes CI failures on Python 3.13 Linux builds
Co-authored-by: sunt05 <1802656+sunt05@users.noreply.github.com>
|
🤖 I've automatically formatted the code in this PR using:
Please pull the latest changes before making further edits. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR builds upon PR #581 (which fixes issue #566) and additionally simplifies the
suews-convertcommand interface while resolving all remaining issues preventing YAML conversion from working properly.Relationship to PR #581
to_yamlmodule issue (No module named supy.cmd.to_yaml #566)Changes (beyond PR #581)
1. Simplified Command Interface
yamloption since 2025+ only supports YAML format2. Fixed Additional YAML Conversion Issues
3. Updated Documentation
input_converter.rst(removed duplicate)transition_guide.rstwith new command syntaxsupy-to-yamland subcommands4. Updated Tests
Usage Examples
Testing
Builds on #581, fixes #566