refid KeyError when adding SNAP "Collocate" node to workflow about pyrosar HOT 5 OPEN

Hmm, I found the relevant XML file in the .pyrosar folder. But I'm not sure where is the code that creates the data in .pyrosar\snap\nodes. Can you point me to where it is @johntruckenbrodt ? I think I can try and sort this out.

from pyrosar.

This is all done in function pyroSAR.snap.auxil.parse_node. It executes gpt Collocate -h, reads the XML representation, and stores it in the .pyrosar/snap/nodes folder.

Setting the node's source(s) is done here. Node.source has a get method (property) and a set method (@source.setter).

So the first question is whether parse_node has to be modified so that the XML representation is better or whether this looks alright and just the source.setter has to be adjusted to properly fill in the values of the list of node IDs you want to set as source.

from pyrosar.

Hi @pbrotoisworo I have never used the Collocate node and apparently pyroSAR is not treating it correctly.
The XMl template looks like this:

<?xml version="1.0" ?><node id="Collocate">	
      	<operator>Collocate</operator>	
      	<sources>		
        		<sourceProduct refid="Read"/>		
        		<slave/>		
        		<sourceProducts/>		
      	</sources>	
      	<parameters class="com.bc.ceres.binding.dom.XppDomElement">		
        		<sourceProductPaths/>		
        		<masterProductName/>		
        		<targetProductType>COLLOCATED</targetProductType>		
        		<renameMasterComponents>true</renameMasterComponents>		
        		<renameSlaveComponents>true</renameSlaveComponents>		
        		<masterComponentPattern/>		
        		<slaveComponentPattern/>		
        		<resamplingType>NEAREST_NEIGHBOUR</resamplingType>		
      	</parameters>	
    </node>

passing multiple IDs as source to a node was developed for SliceAssembly, whose template looks like this:

<?xml version="1.0" ?><node id="SliceAssembly">	
      	<operator>SliceAssembly</operator>	
      	<sources>		
        		<sourceProduct refid="Read"/>		
      		        <sourceProduct.1 refid="Read (2)"/>	
        </sources>	
      	<parameters>		
        		<selectedPolarisations/>		
      	</parameters>	
    </node>

I am sure it is not so hard to adjust, but it will take me some days to find the time for it. If you are faster then a PR is of course always appreciated 😉.

from pyrosar.

Ah I see. Thanks for the pointers I'll take a look at it.

from pyrosar.

Awesome! Thanks @pbrotoisworo

from pyrosar.