Closed m-novikov closed 3 years ago
first of all cool that this works at all. I could confirm with a random h5
file that it first inspects the file and updates the dialog accordingly.
During my test-ride, I found that it doesn't seem to work with outputs from ilastik; I'll send you an example.
edit: this was due to maven problems on my side which I seem to have resolved.
And second observation is the file selection widget - would it be possible to use something more native looking (like the one that is automatically used by scijava org.scijava.widget.FileWidget
?)
This pull request has been mentioned on Image.sc Forum. There might be relevant details there:
https://forum.image.sc/t/combining-swing-form-and-command-parameters/41264/1
Hey @m-novikov, I couldn't get this to load a file via the gui (dataset, axes were detected just fine, but no matter what button I clicked the loading got cancelled).
Hey @m-novikov, I couldn't get this to load a file via the gui (dataset, axes were detected just fine, but no matter what button I clicked the loading got cancelled).
Sorry messed up flag logic, with the last commit it should work
Hi @m-novikov,
it's really cool how you managed to get everything into a single dialog. Macro recording and playback works beautifully and is almost fully compatible with the old implementation. The only pathological case I could come up is directly recording a macro in the old version, and executing this with your PR:
run("Import HDF5", "select=.../sample_data.h5 datasetname=[/test_image: (17, 37, 127, 137, 7) uint8] axisorder=tzyxc");
the "old" plugin recorded the dataset name a little bit differently, including the internal path, shape and data type. I am not sure at the moment how many macros have not removed that information, which will only work if your data doesn't change dimensions (or dtype) over different files, which, to be fair, might often be the case. So I'm not sure users would remove that because it just works. Do you think it would be hard to support this?
this looks good to go - @m-novikov, do you want to add anything else here?
Closes: #37