Open esgomezm opened 10 months ago
Implementing that would be possible (probably rather as part of the save_bioimageio_package
and its related functions).
But allow me to raise the question: When will the DeepImageJ plugin be able to create the required macros on their own from the model description?
Including them explicitly in the package was always meant as a temporary solution...
Hi hi
Implementing that would be possible (probably rather as part of the save_bioimageio_package and its related functions).
I can give it a try if you point me where this is done. I needed it for some zerocost notebook conversion that I just finished.
But allow me to raise the question: When will the DeepImageJ plugin be able to create the required macros on their own from the model description? Including them explicitly in the package was always meant as a temporary solution...
Yes, you're right. JDLL has the functions implemented. I raised it to @arrmunoz already. Tagging @cfusterbarcelo also to keep track of this.
If only a few models are affected I suggest to fix the exported models manually.
If the next spec library release implements this as well, we can use f"{tensor_id}_{macro_file_name}"
as final file name to avoid file name collisions. I have a draft for that already, but ideally we won't need a special deepimagej
section under config
soon and implement the required logic in DeepImageJ instead.
Hi Fynn,
Thank you! As long as JDLL is updated with the specs, you are completely right and that's how it should be. So, do you want me to test the changes in the PR?
well, the above PR isn't ready to be merged, see https://github.com/bioimage-io/spec-bioimage-io/pull/546#discussion_r1449450503
Halo!
When building a model and adding the deepImageJ config with the IJ macros, if the pre and post-processing need to use the same pipeline (e.g., scale range), there is the issue of downloading just one macro rather than two different versions of it (pre+postprocessing).
For example, the following code will download the
scale_linear.ijm
file twice (one for the preprocessing and one for the postprocessing) but with the same name and in the deepImageJ config, both pre and post-processing will point to the same ij macro filescale_linear.ijm
but with the wrong parameters. Could it be possible to rename the downloaded macros accordingly to something likepre_scale_linear.ijm
andpost_scale_linear.ijm
and include as such in the deepImageJ config?Thank you!