[Bug]: mask_block should be added as it is a required argument

[mirzaees]

Checked for duplicates

Yes - I've already checked

Describe the bug

When I was using s1_geocode_slc.py, there was an error related to this argument mask_block that has been changed to a required argument recently but not added to the function using it here:

COMPASS/src/compass/s1_geocode_slc.py

Line 202 in ad3929c

isce3.geocode.geocode_slc(geo_data_blocks=geo_data_blks,

I am not aware of more instances, so I appreciate anybody commenting or sending a PR to fix

Thank you

What did you expect?

I expected [...]

Reproducible steps

just running `s1_cslc.py s1_cslc_geo.yaml ` for any sentinel-1 data

Environment

- Version of this software: compass=0.5.5
- Operating System: Linux
...

[seongsujeong]

Hi @mirzaees, Can you let me know the version of ISCE3 you're using? The issue was reported in the internal ISCE3 GitHub (issue no. 2013) and it should be addressed in the ISCE3 releases from v0.24.0

[mirzaees]

Hi @seongsujeong, I am using version 0.22.1. then I think it should be fixed by now. thank you

[seongsujeong]

No problem! Please let me know if the issue persists even with the recent version of ISCE3.

[scottstanie]

Still an issue @mirzaees ? or close this?