Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suggest to update fetchOrbit.py and topsStack README.md #721

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

yuankailiu
Copy link
Contributor

fetchOrbit.py:

  • Allow reading SLC zip files from a given folder via -d option. So fetchOrbit.py can be used directly as a script to download a batch of orbit files by specifying the SLC folder, rather than doing -i for each input *.zip file.

topsStack README.md:

  • The run_files descriptions in README.md of the topsStack processor are outdated. Here the proper names of each run file is used. Some brief explanations are also updated for each run file.

@chintals
Copy link

Great edit! One brief comment (not related to this PR), the commands in run files for ionospheric estimation are all in serial order (except run_ns+1, subband_and_resamp) regardless of nproc setting. Curious as to why the ns+ files are not parallelized with nproc setting?

@yuankailiu
Copy link
Contributor Author

I don't know. I have always been using HPC cores to do the parallelization of each command line in every run file with stackSentinel.py --num_proc 1.

+ the run_files steps are outdated. The correct steps and explanations are added.

+ added the run_files for correcting ionosphere azimuth burst ramps

+ file path indentation for `ion/` subdirectories
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.

2 participants