Forrest Williams

Results 41 comments of Forrest Williams

Hi @yunjunz I've checked all the executeables, and they complete successfully. I'll start looking into `TimeSeriesAnalysis` class.

Using these pairs as the initial test pairs: | Pair ID | Reference | Secondary | |---------|------------------------------------------|------------------------------------------| | A | LT05_L1TP_061018_19890827_20200916_02_T1 | LT05_L1TP_061018_19890811_20200916_02_T1 | | B | LT05_L1TP_063018_20060925_20200831_02_T1 | LT05_L1TP_063018_20051024_20200901_02_T1...

Sounds good! @jhkennedy your changes look good to me and @alex-s-gardner I'll post some links to examples here when they're ready.

@jhkennedy the above diff looks good to me and appears to match the changes currently in hyp3_autorift [PR #168](https://github.com/ASFHyP3/hyp3-autorift/pull/168).

@jhkennedy, sorry it took me a while to get to this, the the current version looks good to me.

I reviewed the golden tests this morning, and the results are as expected. The sentinel-1 images have the same coverage in test/prod, and the optical images have better coverage in...

Wanted to echo my support for this question as well. Many AI/ML and scientific workflows rely on Python packages that have complex C/C++ dependencies. [GDAL](https://pypi.org/project/GDAL/) is a good example of...

Hey @comma-never-coma, which changes here are due to replacing the SNAPHU source code, and which are to ensure you can successfully compile? Are the changes in the `include` directory the...

Yes, true. we would want to raise an exception if the zip file didn't use zlib compression.

The big hurdle to overcome is allowing `indexed_gzip` to work with other types of DEFLATE streams. [This post](https://dev.to/biellls/compression-clearing-the-confusion-on-zip-gzip-zlib-and-deflate-15g1) does a great job of going over the differences. From my understanding,...