yunjunz commented 1 month ago

Description of proposed changes

docs/dir_structure:
- add example file directory structure and corresponding template input file patterns for isce2/topsStack
- add example template input file patterns for isce2/alosStack & isce2/stripmapStack
modify_network: ignore "," in the input --ex-date(12) options, as copied from the template file

Reminders

[x] Pass Pre-commit check (green)
[x] Pass Codacy code review (green)
[x] Pass Circle CI test (green)
[x] Make sure that your code follows our style. Use the other functions/files as a basis.
[x] If modifying functionality, describe changes to function behavior and arguments in a comment below the function declaration.

Summary by Sourcery

Update documentation with examples for isce2/topsStack and isce2/alosStack, fix comma handling in modify_network script, and enhance logging in view.py.

Bug Fixes:

Fix the handling of commas in the --ex-date(12) options in the modify_network script.

Enhancements:

Change the logging method in the view.py script from verbose print to standard print for map projection messages.

Documentation:

Add example file directory structure and template input file patterns for isce2/topsStack and isce2/alosStack in the documentation.

sourcery-ai[bot] commented 1 month ago

Reviewer's Guide by Sourcery

This pull request adds examples of file directory structure and template input file patterns for isce2/topsStack and isce2/alosStack, as well as modifies the modify_network function to ignore commas in input options. The changes primarily affect documentation and input handling.

Class diagram for modify_network function changes

classDiagram
    class ModifyNetwork {
        - excludeDate: List<String>
        - excludeDate12: List<String>
        + cmd_line_parse(iargs=None)
    }
    ModifyNetwork : +cmd_line_parse(iargs=None)
    ModifyNetwork : +excludeDate = [x.replace(',', '') for x in excludeDate]
    ModifyNetwork : +excludeDate12 = [x.replace(',', '') for x in excludeDate12]

File-Level Changes

Change	Details	Files
Added ionosphere-related file structure and template patterns for topsStack	Added 'ion' and 'ion_dates' directories to the file structure Included ionosphere-related files such as 'filt.ion' and 'raw_no_projection.cor' Added optional ionosphere stack parameters to the template file	`docs/dir_structure.md`
Added ionosphere-related template patterns for alosStack and stripmapStack	Included optional ionosphere stack parameters for KirishimaAlosAT424 Added ionosphere-related file patterns for NCalAlos2DT169	`docs/dir_structure.md`
Modified input handling in modify_network function	Added logic to ignore commas in --ex-date and --ex-date12 input options Updated the processing of excludeDate and excludeDate12 inputs	`src/mintpy/cli/modify_network.py`
Changed print function in check_map_projection	Replaced vprint() with print() for map projection messages	`src/mintpy/view.py`

Tips and commands

#### Interacting with Sourcery - **Trigger a new review:** Comment `@sourcery-ai review` on the pull request. - **Continue discussions:** Reply directly to Sourcery's review comments. - **Generate a GitHub issue from a review comment:** Ask Sourcery to create an issue from a review comment by replying to it. - **Generate a pull request title:** Write `@sourcery-ai` anywhere in the pull request title to generate a title at any time. - **Generate a pull request summary:** Write `@sourcery-ai summary` anywhere in the pull request body to generate a PR summary at any time. You can also use this command to specify where the summary should be inserted. #### Customizing Your Experience Access your [dashboard](https://app.sourcery.ai) to: - Enable or disable review features such as the Sourcery-generated pull request summary, the reviewer's guide, and others. - Change the review language. - Add, remove or edit custom review instructions. - Adjust other review settings. #### Getting Help - [Contact our support team](mailto:support@sourcery.ai) for questions or feedback. - Visit our [documentation](https://docs.sourcery.ai) for detailed guides and information. - Keep in touch with the Sourcery team by following us on [X/Twitter](https://x.com/SourceryAI), [LinkedIn](https://www.linkedin.com/company/sourcery-ai/) or [GitHub](https://github.com/sourcery-ai).

codeautopilot[bot] commented 1 month ago

PR summary

This Pull Request updates the documentation to include example directory structures and template input file patterns for isce2/topsStack and isce2/alosStack, enhancing clarity for users setting up these stacks. It also fixes a bug in the modify_network.py script by improving the handling of commas in the --ex-date(12) options, ensuring smoother user input processing. Additionally, it modifies the logging method in view.py to use standard print statements for map projection messages, which may improve readability and consistency in output.

Suggestion

Consider adding unit tests for the modify_network.py script to ensure that the handling of commas in the --ex-date(12) options is robust and functions as expected. This could prevent future regressions and improve code reliability. Additionally, ensure that the documentation changes are reflected in any related user guides or tutorials to maintain consistency across all user-facing materials.

insarlab / MintPy