Skip to content

Aperture Tool Bugs #23

@leroygorham

Description

@leroygorham

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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions