-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
displaying the correct area #74
Comments
To better address this I have prepared 2 datasets and found that So it would be good to support both options so I can carefully try which one works better and then we set this one as default.
/data/HDF5EOS/subset0unittestGalapagosSenDT128/mintpy
|
Is it possible to visualize these using mintpy viewer? That way we can choose the right attribute and I can add it to the site. |
Yes, absolutely. If we have a S1* file in radar coordinates it is easy. The first/last column/row values give the lat/longs of the edges. If the S1* file is in geo-coordinates it is more complicated. We probably should take the smallest longitude/latitude largest latitude/longitude but it may be difference for descending (southward traveling) and ascending (northward traveling) orbits (sensor always looks to right). - geometry file in radar coordinates - S1 file in geo-coordinates:*
- S1 file in radar coordinates (Miami):*
|
So the mintpy library has no way of extracting this? Right now we just use scene_footprint and data_footprint. |
How do we currently do it: Are we using
|
I've fixed the ingest script so it ingests the mintpy.subset.Lalo attribute if it's there. Can you ingest a couple of datasets so I can test? I'd do it but I will be busy until I get home around 7 pm today. |
I re-ingested three datasets. I hope this worked. I used the version if you do s.bop 129.213.120.104/start/25.78/-80.3/11.0?flyToDatasetCenter=true&startDataset=CSK_SM_091_0000_20180916_XXXXXXXX_N19368_N19540_W155666_W155481 129.213.120.104/start/25.78/-80.3/11.0?flyToDatasetCenter=true&startDataset=S1_IW3_048_0081_0082_20150921_20211112_SM_PS 129.213.120.104/start/25.78/-80.3/11.0?flyToDatasetCenter=true&startDataset=S1_IW3_048_0081_0082_20150921_20211112_SM |
I only see subset Lalo in 1 area (one that I ingested when I first sent the email). Did you git pull? Maybe that’s why the ones you did didn’t get infested.
Alfredo
…Sent from my iPhone
On Aug 16, 2022, at 10:44 AM, Falk Amelung ***@***.***> wrote:
I re-ingested three datasets. I hope this worked. I used the version if you do s.bop
129.213.120.104/start/25.78/-80.3/11.0?flyToDatasetCenter=true&startDataset=CSK_SM_091_0000_20180916_XXXXXXXX_N19368_N19540_W155666_W155481
129.213.120.104/start/25.78/-80.3/11.0?flyToDatasetCenter=true&startDataset=S1_IW3_048_0081_0082_20150921_20211112_SM_PS
129.213.120.104/start/25.78/-80.3/11.0?flyToDatasetCenter=true&startDataset=S1_IW3_048_0081_0082_20150921_20211112_SM
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.
|
I re-ingested another one in the case I did something stupid. Please try those two: 129.213.120.104/start/25.78/-80.3/11.0?flyToDatasetCenter=true&startDataset=CSK_SM_091_0000_20180916_XXXXXXXX_N19368_N19540_W155666_W155481 |
Can you try? Might be working. Didn't completely understand your message. It is pretty clear, but before we had more logic for determining subsets. Not sure if the above will break it. |
|
Yes, I undid the changes last night. I have fixed it now so that the grouping is still there. Can you check and let me know if it's correct? Also, "S1_IW3_048_0081_0082_20191111_20220710_PS" has mintpy.subset.lalo yet it has no data_footprint. *Edit: Oh, I realize this is a high res file as you mentioned above. |
Yes, the grouping looks good again. Thank you! I looked a bit at the code for data_footprint: I would think that it is possible to get the data_footprint from the min and max from the lat/long arrays. Maybe we should first convince ourselves that it works, and before doing a PR to mintpy we ask Yunjun whether he would accept it. |
If the grouping looks good and the new logic looks good as well, can we close this issue and open a new one to add data_footprint to save_hdfeos5? |
yes, sure. |
I don't remember. I agree we shouldn't change it if it's working. Closing this and will open new issue for modifying the mintpy script. |
I don't remember. I agree we shouldn't change it if it's working. Closing this and will open new issue for modifying the mintpy script in the mintpy repo. |
I think it should work. @yunjunz what do you think? Should I open a new issue and implement this? |
I did not read all comments in this issue, as it's pretty long.
If "this" means "specifying the data_footprint for datasets in the radar coordinates", yes, it makes sense to me, as we are now ingesting radar-coded products. Please go for it. |
When a subset is processed the entire frame is shown in the overview window. Would it be possible to show only the area of processed? Here two examples:
1st example (Mauna Loa):
https://insarmaps.miami.edu/start/19.6610/-154.7935/8.0000?flyToDatasetCenter=false&startDataset=CSK_SM_091_0000_20180916_XXXXXXXX_N19368_N19540_W155666_W155481
I think we need to check for the existence of the mintpy.subset.lalo attribute (and miaplypy.subset.lalo which has preference)) options and use if given:
Second example (Miami):
![image](https://user-images.githubusercontent.com/11162588/182533558-f9877f8a-133f-4708-8823-a0375cc79703.png)
![image](https://user-images.githubusercontent.com/11162588/182534419-75674422-84f4-4e91-8b41-d766b48d89db.png)
129.213.120.104/start/25.78/-80.3/14.0?flyToDatasetCenter=true&startDataset=S1_IW3_048_0081_0082_20150921_20211112_SM_PS
Other notes:
Occasionally the subset area is properly displayed. This could be the
mintpy.save.hdfEos5.subset = yes
option when creating the S1* files, but I need to find an example to check. See this example for a blue subset box:The text was updated successfully, but these errors were encountered: