Tags: isce-framework/s1-reader
Tags
Patch for "no polynomial" case for azimuth FM rate mitigation & handl… …ing the burst without burst polygon (#134) * Take care of the case when the az. time of the polynomial list does not cover sensing start / stop * Let `InterpolatedUnivariateSpline` to return boundary values when the input parameter is out of the interpolator's range * extrapolate the burst center point the border polygon when the burst polygon data is missing in the metadata * Interface to turn on / off the correction * ditch out burst border polygon extrapolation, and fill that with `None` * Place empty geometry instead of `None` * whitespace removal * Apply suggestions from code review Co-authored-by: Liang Yu <liangjyu@gmail.com> * add description about "no polynomials for burst" case * get rid of unnecessary module in `s1-reader.py` * bump the version * more clarification about the missing polynomial case --------- Co-authored-by: Seongsu Jeong <seongsu.jeong@jpl.nasa.gov> Co-authored-by: Liang Yu <liangjyu@gmail.com>
Option to concatenate RESORB file (#133) * s1_orbit.py: add function to concatenate two orbit files to make a larger one This allows us to extend the time span for RESORB products * fix the output name and stop time parse. return the outfile * add tests for RESORB edge case and concatenation function * rename the pytest for the function * working version of RESORB concat, with overlap and XML order issue solved * load OSVs for ANX time crossing without ISCE3 module * code cleanup * code cleanup * notes about concatenation order; option to concatenate RESORB file in the file list; updates on unit test and test data * remove bad return value from docstring * fix on padding at tail of the timeframe * Apply suggestions from code review Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * reference update to `_covers_timeframe` * make RESORB concatenation optional * s1_annotation import style change * extracting the OSv list merging routine as `merge_osv_list()`; leave a note on OSV list sorting routine * declaring the orbit time padding as `PADDING_TIME_ORBIT`; option to take the list of orbit files when loading OSV lists * unit test update * RESORB combination search attempt into `get_orbit_file_from_list` * revision on the note regarding the orbit concatenation * fix on computing relative time wrt. `datetime_orbit_ref` * RESORB concatenation is optional for `retrieve_orbit_file` and `False` by default * variable name change; logic fix on checking RESORB pair search result; take care of case when the RESORB pair search failed * Apply suggestions from code review Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> * docstring revised * soft-coding the orbit file name slicing index in `_generate_filename`; comment revision on `test_orbit.py` * Example orbit file name for the reference * early return * Apply suggestions from code review Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: Heresh Fattahi <hersh.fattahi@gmail.com> * variable renaming for clarify; sort the input orbit orbit list * revise comment in `get_resorb_pair_from_list()` * Ditching out the orbit file order check * fix on s1_orbit.py * get rid of the duplicated 30 min of padding in `get_orbit_dict()` * variable name revision * description about short search time window; revision about the comments. * Note about timing design of RESORB * Apply suggestions from code review Co-authored-by: Liang Yu <liangjyu@gmail.com> * import `PADDING_SHORT` from `s1_orbit.py` * docstring revised * docstring revision; revision on if statementl docstring for `merge_osv_list()` * renmove unnecessary comment * `_get_uct_time_from_osv` to exxtract OSV's UTC time * adjustment in module import order * revised docstring for `get_orbit_dict`; get rid of variavles `pad*_time` * Apply suggestions from code review Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * leading underscore names for non-public functions * bump the version --------- Co-authored-by: Scott Staniewicz <scott.j.staniewicz@jpl.nasa.gov> Co-authored-by: Seongsu Jeong <seongsu.jeong@jpl.nasa.gov> Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> Co-authored-by: Heresh Fattahi <hersh.fattahi@gmail.com> Co-authored-by: Liang Yu <liangjyu@gmail.com>
Release updates (#125) * bump version to 0.2.0 for new release * add missing lxml to environment.yaml, pin minimum versions * use env file instead of hard coding installs for `freeze-deps.sh` * update readme for install instructions * specfile script needs `--network=host`. `conda env` has no install or -y * point README install instrs. to correct file * use geoffs version of dependency freezing script * remove brackets from license * Update .circleci/freeze-deps.sh Co-authored-by: Seongsu Jeong <sjeong.kr@gmail.com> --------- Co-authored-by: Seongsu Jeong <sjeong.kr@gmail.com>
Beta release compass (#94) * Replacing the loaders in Burst* class into class methods, with further implementation for thermal and EAP correction * fix on determining beta_naught; addressing PEP8 issues * Bug fix and feature addition to BurstEAP; restructuring LUT exportation * Readibility improvement; removing unnecessary imports * Format change on `burst_id`; keeping the absolute orbit number inside `Sentinel1BurstSlc` * updates on test_bursts.py * keeping the basename of the CADS and NADS for populating RTC metadata * Update src/s1reader/s1_annotation.py Readability improvement on equation Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_annotation.py Removing commented out code Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_annotation.py Reverting the docstring to be split into two lines for PEP8 compliance Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_annotation.py Improving docstring of the code copied from isce2 Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Removing the commented out codes Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py improvement on code brevity Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py renaming variable for better clarity Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py renaming variable name for clarity Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py variable name revised for clarity Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py variable renamed for clarity Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py improvement on docstring Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Readability improvement of equation Co-authored-by: Liang Yu <liangjyu@gmail.com> * addressing comments bt @LiangJYu * docstring fix; variables renamed for clarity * implemented s1_annotation.AucCal.load_from_zip_file() * readability improvement * s1_annotation.py - code cleanup; excention handling for AUX_CAL; PEP8 compliance * docstring for `s1_burst_slc.eap_compensation_lut()` * class import scheme changed * PEP8 compliance * initial structure of modification * less mod compared to the branch fork * extended coeffs for FM rate and Doppler centroid to be used for azimuth FM mismatch rate mitigation * Working version of azimuth FM rate mismatch mitigation * Update src/s1reader/s1_orbit.py Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> * Update src/s1reader/s1_orbit.py Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> * Update src/s1reader/s1_orbit.py Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> * Update src/s1reader/s1_orbit.py Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> * Update src/s1reader/s1_orbit.py Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> * Update src/s1reader/s1_orbit.py Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> * addressing comments from @gshiroma * trailing white spaces are removed * docstring updated * Code cleanup and docstrings * code cleanup * trailing white space removed * docsrting added; variable renaming for less verbosity; * code cleanup and simplification * variable name simplification * clde cleanup * Output as isce3.core.LUT2d * threshold and num_iter for rdr2geo * Addressing codacy issue * Trailing whitespace removed * codacy issue * docstring clarification on `BurstExtendedCoeffs` * whitespace removed * Update src/s1reader/s1_orbit.py Co-authored-by: vbrancat <virginia.brancato@jpl.nasa.gov> * Update src/s1reader/s1_orbit.py Co-authored-by: vbrancat <virginia.brancato@jpl.nasa.gov> * Update src/s1reader/s1_annotation.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * comment added for clarity * Update src/s1reader/s1_annotation.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_annotation.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_annotation.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * mod on the suggested commit to reuse the topo output filenames * fix on variable * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_annotation.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: vbrancat <virginia.brancato@jpl.nasa.gov> * Update src/s1reader/s1_burst_slc.py Co-authored-by: vbrancat <virginia.brancato@jpl.nasa.gov> * typo fix * Update src/s1reader/s1_burst_slc.py Co-authored-by: vbrancat <virginia.brancato@jpl.nasa.gov> * Re-using the parsed polynomials; mitigation results as ISCE3 LUT2d * revision on import format * TODO removed * Option to turn on / off the functionalities for az fm rate mismatch mitigation * removing alias for `Rdr2Geo` * whitespace removed after resolving conflict * addressing codacy issue * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_annotation.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_annotation.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_reader.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * variable name changed * bug fixed; variable name updated * addressing codacy issues * codacy issue * variable name changed * pulling out functions that does not need to be methods * whitespace removed * updates after linting * Docstring for `_burst_from_safe_dir` revised * Docstring for `_burst_from_zip` revised * Docstring for `load_bursts` revised * Docstring for `burst_from_xml` revised * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Docstring for `az_fm_rate_mismatch_mitigation` revised * Use temporary directory when `path_scratch` was not provided; take in az/rg steps as options to define radargrid * codacy issue * docstring updated; variable name changed * unnecessary members removed; docstring revised * temporary filename format changed * Update src/s1reader/s1_annotation.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Update src/s1reader/s1_annotation.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Update src/s1reader/s1_annotation.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Update src/s1reader/s1_annotation.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * unit_degree -> in_degree * Update src/s1reader/s1_burst_slc.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Apply suggestions from code review Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * variable name format revised * `mitigate_az_fmrate_mismatch` removed * PEP8 issues * Handle the case that the burst's sensing period exceeds the range of `polynomial_list` * version.py updated Co-authored-by: Seongsu Jeong <seongsu.jeong@jpl.nasa.gov> Co-authored-by: Liang Yu <liangjyu@gmail.com> Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> Co-authored-by: vbrancat <virginia.brancato@jpl.nasa.gov> Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com>
FM rate mismatch - follow-up bug fix (#92) * Replacing the loaders in Burst* class into class methods, with further implementation for thermal and EAP correction * fix on determining beta_naught; addressing PEP8 issues * Bug fix and feature addition to BurstEAP; restructuring LUT exportation * Readibility improvement; removing unnecessary imports * Format change on `burst_id`; keeping the absolute orbit number inside `Sentinel1BurstSlc` * updates on test_bursts.py * keeping the basename of the CADS and NADS for populating RTC metadata * Update src/s1reader/s1_annotation.py Readability improvement on equation Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_annotation.py Removing commented out code Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_annotation.py Reverting the docstring to be split into two lines for PEP8 compliance Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_annotation.py Improving docstring of the code copied from isce2 Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Removing the commented out codes Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py improvement on code brevity Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py renaming variable for better clarity Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py renaming variable name for clarity Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py variable name revised for clarity Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py variable renamed for clarity Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py improvement on docstring Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Readability improvement of equation Co-authored-by: Liang Yu <liangjyu@gmail.com> * addressing comments bt @LiangJYu * docstring fix; variables renamed for clarity * implemented s1_annotation.AucCal.load_from_zip_file() * readability improvement * s1_annotation.py - code cleanup; excention handling for AUX_CAL; PEP8 compliance * docstring for `s1_burst_slc.eap_compensation_lut()` * class import scheme changed * PEP8 compliance * initial structure of modification * less mod compared to the branch fork * extended coeffs for FM rate and Doppler centroid to be used for azimuth FM mismatch rate mitigation * Working version of azimuth FM rate mismatch mitigation * Update src/s1reader/s1_orbit.py Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> * Update src/s1reader/s1_orbit.py Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> * Update src/s1reader/s1_orbit.py Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> * Update src/s1reader/s1_orbit.py Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> * Update src/s1reader/s1_orbit.py Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> * Update src/s1reader/s1_orbit.py Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> * addressing comments from @gshiroma * trailing white spaces are removed * docstring updated * Code cleanup and docstrings * code cleanup * trailing white space removed * docsrting added; variable renaming for less verbosity; * code cleanup and simplification * variable name simplification * clde cleanup * Output as isce3.core.LUT2d * threshold and num_iter for rdr2geo * Addressing codacy issue * Trailing whitespace removed * codacy issue * docstring clarification on `BurstExtendedCoeffs` * whitespace removed * Update src/s1reader/s1_orbit.py Co-authored-by: vbrancat <virginia.brancato@jpl.nasa.gov> * Update src/s1reader/s1_orbit.py Co-authored-by: vbrancat <virginia.brancato@jpl.nasa.gov> * Update src/s1reader/s1_annotation.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * comment added for clarity * Update src/s1reader/s1_annotation.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_annotation.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_annotation.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * mod on the suggested commit to reuse the topo output filenames * fix on variable * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_annotation.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: vbrancat <virginia.brancato@jpl.nasa.gov> * Update src/s1reader/s1_burst_slc.py Co-authored-by: vbrancat <virginia.brancato@jpl.nasa.gov> * typo fix * Update src/s1reader/s1_burst_slc.py Co-authored-by: vbrancat <virginia.brancato@jpl.nasa.gov> * Re-using the parsed polynomials; mitigation results as ISCE3 LUT2d * revision on import format * TODO removed * Option to turn on / off the functionalities for az fm rate mismatch mitigation * removing alias for `Rdr2Geo` * whitespace removed after resolving conflict * addressing codacy issue * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_annotation.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_annotation.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Update src/s1reader/s1_reader.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * variable name changed * bug fixed; variable name updated * addressing codacy issues * codacy issue * variable name changed * pulling out functions that does not need to be methods * whitespace removed * updates after linting * Docstring for `_burst_from_safe_dir` revised * Docstring for `_burst_from_zip` revised * Docstring for `load_bursts` revised * Docstring for `burst_from_xml` revised * Update src/s1reader/s1_burst_slc.py Co-authored-by: Liang Yu <liangjyu@gmail.com> * Docstring for `az_fm_rate_mismatch_mitigation` revised * Use temporary directory when `path_scratch` was not provided; take in az/rg steps as options to define radargrid * codacy issue * docstring updated; variable name changed * unnecessary members removed; docstring revised * temporary filename format changed * Update src/s1reader/s1_annotation.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Update src/s1reader/s1_annotation.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Update src/s1reader/s1_annotation.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Update src/s1reader/s1_annotation.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * unit_degree -> in_degree * Update src/s1reader/s1_burst_slc.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Update src/s1reader/s1_burst_slc.py Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * Apply suggestions from code review Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com> * variable name format revised * `mitigate_az_fmrate_mismatch` removed * PEP8 issues * Handle the case that the burst's sensing period exceeds the range of `polynomial_list` Co-authored-by: Seongsu Jeong <seongsu.jeong@jpl.nasa.gov> Co-authored-by: Liang Yu <liangjyu@gmail.com> Co-authored-by: Gustavo H. X. Shiroma <52007211+gshiroma@users.noreply.github.com> Co-authored-by: vbrancat <virginia.brancato@jpl.nasa.gov> Co-authored-by: Scott Staniewicz <scott.stanie@gmail.com>
PreviousNext