-
Notifications
You must be signed in to change notification settings - Fork 98
Description
There are some minor bugs in the Aperture Tool.
https://github.com/ngageoint/MATLAB_SAR/blob/master/Tools/ApertureTool/ApertureTool.m#L1459
This does not conform to the SICD standard, there is another field name called "WindowName" that needs to be added:
meta.Grid.Col.WgtType.WindowName = WgtType;
https://github.com/ngageoint/MATLAB_SAR/blob/master/Tools/ApertureTool/ApertureTool.m#L1460
Same as line 1459, this needs to be changed to::
meta.Grid.Row.WgtType.WindowName = WgtType;
https://github.com/ngageoint/MATLAB_SAR/blob/master/Tools/ApertureTool/ApertureTool.m#L1454
There is a typo on this line: "CosOnPed" should be "CosinePed"
elseif get(handles.CosinePed,'Value')
Also, this line causes an error with Umbra data as they use the optional field "meta.ImageFormation.Processing.Type" as a structure and not as a string:
https://github.com/ngageoint/MATLAB_SAR/blob/master/Tools/ApertureTool/ApertureTool.m#L1419
This line appears to serve no purpose except as information only, so perhaps this line could be removed.