docs/dir_structure: add iono example for topsStack & alosStack

[yunjunz]

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

• Pass Pre-commit check (green)
• Pass Codacy code review (green)
• Pass Circle CI test (green)
• Make sure that your code follows our style. Use the other functions/files as a basis.
• 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]

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]

Loading

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 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 for questions or feedback.
• Visit our documentation for detailed guides and information.
• Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

[codeautopilot]

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.

Disclaimer: This comment was entirely generated using AI. Be aware that the information provided may be incorrect.

Current plan usage: 0.00%

Have feedback or need help?
Discord
Documentation
[email protected]

[sourcery-ai]

Hey @yunjunz - I've reviewed your changes - here's some feedback:

Overall Comments:

• The changes in view.py from vprint() to print() might affect verbosity control. Please review if this change is intentional and consider its potential impact on the program's behavior.

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟡 Security: 1 issue found
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}