GNSS source and outlier removal updates pt2

[rzinke]

.~~~~~~. ARIA_prep.ipynb 1.1 Download GUNW Interferograms

• Added "" around password to escape special characters

  .~~~~~~. Secular_Requirement_Validation.ipynb 3.2 Tropospheric Delay Correction

• Added statement to read my_sites file for tropo flag

4.1 Estimate InSAR LOS Velocities

• Added sentences to end of text block describing the basis functions to be solved for, and that those same functions will be applied to both InSAR and GNSS datasets
• Added cell defining basis functions
• Changed MintPy call to solve for linear fits using user-defined basis functions
• Added plot of linear velocity for user reference (was this removed intentionally?)

4.3 Find Collocated GNSS Stations

• fixed typo, clarified text, and specified GNSS source options
• Added statement to read my_sites file for GNSS argument
• gnss_source now passed to search_gnss function

4.4 Get GNSS Position Time Series

• gnss_source now passed to get_gnss_class function

4.5 Estimate GNSS LOS Velocities

• Added functions for iteratively removing outliers and estimating standard errors

A.4 GNSS Timeseries Plots

• gnss_source now passed to get_gnss_class

[review-notebook-app[bot]]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[review-notebook-app[bot]]

View / edit / reply to this conversation on ReviewNB

EJFielding commented on 2024-09-07T14:44:36Z ----------------------------------------------------------------

The new table output and plots look great!

review-notebook-app[bot] commented 2 months ago

View / edit / reply to this conversation on ReviewNB

EJFielding commented on 2024-09-07T14:44:37Z ----------------------------------------------------------------

Line #145.    gnss_velocities = []

I would recommend breaking this into a separate cell here. It is hard to see where the beginning of the regular code starts after the long set of function definitions.

review-notebook-app[bot] commented 2 months ago

View / edit / reply to this conversation on ReviewNB

EJFielding commented on 2024-09-07T14:44:38Z ----------------------------------------------------------------

Can this set of plots be changed to use the GNSS data after the outliers were removed?

• © Githubissues.
• Githubissues is a development platform for aggregating issues.

[review-notebook-app[bot]]

View / edit / reply to this conversation on ReviewNB

EJFielding commented on 2024-09-07T14:44:37Z ----------------------------------------------------------------

Line #145.    gnss_velocities = []

I would recommend breaking this into a separate cell here. It is hard to see where the beginning of the regular code starts after the long set of function definitions.

review-notebook-app[bot] commented 2 months ago

View / edit / reply to this conversation on ReviewNB

EJFielding commented on 2024-09-07T14:44:38Z ----------------------------------------------------------------

Can this set of plots be changed to use the GNSS data after the outliers were removed?

• © Githubissues.
• Githubissues is a development platform for aggregating issues.

[review-notebook-app[bot]]

View / edit / reply to this conversation on ReviewNB

EJFielding commented on 2024-09-07T14:44:38Z ----------------------------------------------------------------

Can this set of plots be changed to use the GNSS data after the outliers were removed?

• © Githubissues.
• Githubissues is a development platform for aggregating issues.