search

opera-adt / COMPASS

COregistered Multi-temPorAl Sar Slc
Apache License 2.0
39 stars 18 forks source link

Placeholder for RFI information #115

Closed seongsujeong closed 1 year ago

seongsujeong commented 1 year ago

This PR adds placeholder for RFI information for CSLC output HDF5 file. The structure of the metadata is based on the pending PR in s1-reader for RFI reader here

seongsujeong commented 1 year ago

Can we coordinate with @LiangJYu and find a better place for them? Thanks

@LiangJYu Based on #101, I think we write out the QA information into .json file, whose path is out_paths.stats_json_path?

LiangJYu commented 1 year ago

Can we coordinate with @LiangJYu and find a better place for them? Thanks

@LiangJYu Based on #101, I think we write out the QA information into .json file, whose path is out_paths.stats_json_path?

@seongsujeong What do you think about me including the RFI fields in the PR in to #101?

seongsujeong commented 1 year ago

What do you think about me including the RFI fields in the PR in to https://github.com/opera-adt/COMPASS/pull/101?

@LiangJYu Thanks for the suggestion. I think that would be the fastest way to add the placeholder for RFI. The fields for RFI is defined in the list rfi_items in h5_helpers.py in here. Please let me know if you need more information.

LiangJYu commented 1 year ago

What do you think about me including the RFI fields in the PR in to #101?

@LiangJYu Thanks for the suggestion. I think that would be the fastest way to add the placeholder for RFI. The fields for RFI is defined in the list rfi_items in h5_helpers.py in here. Please let me know if you need more information.

Closing this PR. #101 updated to include RFI