DDMAL / pixel_wrapper

Rodan Pixel.js Wrapper
Other
0 stars 1 forks source link

Include selectRegionLayer when exporting to Rodan #29

Closed EricHanLiu closed 6 years ago

EricHanLiu commented 6 years ago

Will always be the last layer in the list of layers (so last output port also).

The user has to create 1 more output port than input port. It was difficult to have this as a mandatory port since it has ID -1, and I couldn't make an output port with layer ID -1 properly

Updated README with instructions

vigliensoni commented 6 years ago

Could you please add more details to the documentation of classification (https://github.com/DDMAL/pixel_wrapper/blob/cde1347b014295bb7c17be6077f276b2af606cf2/README.md#classification) for what the extra port is needed?

EricHanLiu commented 6 years ago

Will do

EricHanLiu commented 6 years ago

Done

I originally thought that skipping output port numbers (ie. setting the outputs as Layer 1, Layer 5, Layer 7) would work properly, but it turns out I was mistaking it for the input ports. The input ports can be in an arbitrary order, but unfortunately the output ports have to be in numerical/incremental order. Otherwise Rodan crashes saying The job did not produce the output file for ....

I tried setting the input and output ports to the same arbitrary layer numbers (so input 1, 3, 5, output 0, 1, 3, 5, 2, where 2 is for the selectRegionLayer. I've moved the 2 port to other ports as well, but it seems like Rodan needs the outputs to be increasing without leaps, and it doesn't care about input/output matching.