Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

does not overwrite output and throws no error #85

Open
tischi opened this issue Aug 6, 2021 · 3 comments
Open

does not overwrite output and throws no error #85

tischi opened this issue Aug 6, 2021 · 3 comments

Comments

@tischi
Copy link
Contributor

tischi commented Aug 6, 2021

@wolny @k-dominik

When running such a IJ macro command:

run("Export HDF5", "select=&outputPath exportpath=&outputPath datasetname=data compressionlevel=0 input=resampled");

My observation is that ilastik does not overwrite files if outputPath does exist already; and it also does not throw an error (which makes it confusing). Can you confirm this? And should we change this?

@k-dominik
Copy link
Contributor

Hey @tischi,

thx for reporting. I think in general we should allow people to put multiple datasets in their hdf5 files. So warning just based on outputpath's existence is not sufficient. But we could warn for overwriting the same dataset in a file, maybe?

@tischi
Copy link
Contributor Author

tischi commented Aug 6, 2021

we should allow people to put multiple datasets in their hdf5 files

do you think it should currently overwrite an existing dataset?

@k-dominik
Copy link
Contributor

I'd expect it to overwrite, yes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants