Remove the save_result from the main UDP#609
Conversation
|
@HansVRP / @Pratichhya - Feel free to share your thoughts on how we can best approach this. Right now it is quite unclear that the user needs to add this The issue was raised by the BiaB integration where they simply read out a UDP definition from the APEx Algorithm Catalogue in an automated way, so there is no real way for them figuring out whether to add this parameter to the |
🔍 Catalogue's Preview Site DeployedYour changes have been deployed to the preview site: 🔗 Preview URL: https://esa-apex.github.io/apex-algorithms-catalogue-web/pr-preview/pr-609/ This preview will be updated automatically when you push new changes to your PR. |
|
@JanssenBrm @Pratichhya I would be fine with removing the save step if you can keep the metadata somehow in the final product the user downloads (the metadata is compartible with Gtif and NetCDF.... zar not tested) |
|
From the user side, I think @JanssenBrm already implemented an alternative approach for handling metadata in the notebook workflow here: To further investigate how to update the output metadata outside the save_result node while still ensuring it is preserved in the final downloaded product (e.g., GTiff and NetCDF outputs), it might be worth creating a separate issue to explore metadata handling: #611 |
In the current implementation, the
save_resultstep is included in the full UDP definition. Whenever a user uses theexecute_batchfunction without changes, it always downloads 2 files. This is causing the following confusion:auto_add_save_result=False, which requires them to either look at the UDP or only realise (if they do) after executing the service. It is not documented, except for the notebook example.execute_batchthe GTIFF is downloaded anyway.Nevertheless, I do see the usefulness of adding the metadata to the output files and maybe there is a workaround for this.