Closed kvantricht closed 2 months ago
- Before trying on a different location I would like to test it with the files that you used here.
@GriffinBabe in fact I don't think these are very useful right now. Bands are still only ['classification', 'probability']
. We need updated ones with all probabilities right?
is save_result()
now always triggered when requesting postprocessing? I can imagine e.g. for the demo that users just want the end product. The only use case for saving intermediate results is probably the global processing where we still want to play with the probabilities afterwards.
Let's make it parametrizable in generate_map
with default=False?
@kvantricht
generate_map
function to connect to the backdoor URL if the users wants to connect to CDSECropTypeClassifier
to determine the number of output bands to output and specify in the computing graph, now I do it outside the UDF, but I also pass it to the PostProcessor
so it can reclassify with the correct labels instead of the argmax indices.
@kvantricht Made some good progress on this issue today
A few things to discuss:
np.argmax(...)
. However, we do not have access to a LUT as in the catboost inference. One way could be to read this information from the catboost model outside the UDFs in thegenerate_map
function and then provide the LUT to all UDFs from the parameters, I propose we discuss this tomorrow. (At the moment I'm simply leaving the argmax indices as new labels, which is consistent with the default catboost model we provide, but is not future proof)