Merge pull request #9 from ValkyrieSystems/bugfix-from-xml-apc-handling

Bugfix metadata params from xml

sarkit/standards/sicd/projection/params.py

@@ -28,6 +28,7 @@ def _get_rcv_poly(xmlhelp):
     rcv_apc_polys = [
         xmlhelp.load(f"{{*}}Position/{{*}}RcvAPC/{{*}}RcvAPCPoly[@index='{c}']")
         for c in rcv_apc_indices
+        if c is not None
     ]
     if rcv_apc_polys:
         return functools.reduce(npp.polyadd, rcv_apc_polys) / len(rcv_apc_polys)

sarkit/verification/sicd_consistency.py

@@ -1753,9 +1753,6 @@ def check_rcvapcindex(self) -> None:
                 "./{*}RadarCollection/{*}RcvChannels/{*}ChanParameters/{*}RcvAPCIndex"
             )
         ]
-        with self.need("RcvAPC and RcvAPCIndex must be included together"):
-            assert len(rcvapc_indices) > 0
-            assert len(rcvapcindex) > 0
 
         for index in rcvapcindex:
             with self.need(

tests/standards/sicd/test_projection.py

@@ -46,6 +46,14 @@ def test_metadata_params():
     assert not unset_attrs
 
 
+def test_metadata_params_without_optional_apcs():
+    etree = lxml.etree.parse(DATAPATH / "example-sicd-1.3.0.xml")
+    assert ss_proj.MetadataParams.from_xml(etree).Rcv_Poly is not None
+    for el in etree.findall(".//{*}RcvAPCIndex") + etree.findall(".//{*}RcvApcPoly"):
+        el.getparent().remove(el)
+    assert ss_proj.MetadataParams.from_xml(etree).Rcv_Poly is None
+
+
 def test_metadata_params_is_monostatic(example_proj_metadata):
     example_proj_metadata.Collect_Type = "MONOSTATIC"
     assert example_proj_metadata.is_monostatic()

tests/verification/test_sicd_consistency.py

@@ -676,3 +676,29 @@ def test_check_image_formation_timeline(sicd_con):
     )
     sicd_con.check("check_image_formation_timeline")
     assert sicd_con.failures()
+
+
+def test_check_rcvapcindex(sicd_con):
+    # Invalid APCIndex
+    sicd_con.xmlhelp.set(
+        "./{*}RadarCollection/{*}RcvChannels/{*}ChanParameters/{*}RcvAPCIndex",
+        100,
+    )
+    sicd_con.check("check_rcvapcindex")
+    assert sicd_con.failures()
+
+    # No APCIndex with APCPolys is OK
+    rcvapcindex = sicd_con.sicdroot.find(
+        "./{*}RadarCollection/{*}RcvChannels/{*}ChanParameters/{*}RcvAPCIndex"
+    )
+    rcvapcindex.getparent().remove(rcvapcindex)
+    sicd_con.check("check_rcvapcindex")
+    assert not sicd_con.failures()
+
+
+def test_check_rcvapcindex_nopolys(sicd_con):
+    # APCIndex with no APCPolys
+    rcvapcnode = sicd_con.sicdroot.find("./{*}Position/{*}RcvAPC")
+    rcvapcnode.getparent().remove(rcvapcnode)
+    sicd_con.check("check_rcvapcindex")
+    assert sicd_con.failures()