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]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[review-notebook-app]

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]

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]

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?

[EJFielding]

I made some suggestions with ReviewNB for two small adjustments, but I am approving it as-is to get it merged now.