-
Notifications
You must be signed in to change notification settings - Fork 192
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
Import / Export without empty directories #3764
Comments
@sphuber Let's discuss whether it perhaps makes sense to - until the repo migration - add a "lazy" node folder creation that only creates node folders when something should be put inside. |
I agree - and it should be a fairly easy(TM) fix in the end. I can try and knock it out. |
One comment by @sphuber : Since we are currently not tracking the content of folders, this change would mean that we are no longer able to detect whether the folder was (e.g. accidentally) deleted. I.e. we should weigh whether the reduction in inodes is worth giving up this consistency check (which is of course quite basic - e.g. missing files are anyhow not detected). Perhaps @giovannipizzi can comment |
This will be addressed by the new archive format aiidateam/AEP#21 |
Heya, I've marked this a being closed by: #3445.
I very much doubt anyone's going to have time to do this now, it'll just be wasted effort |
For some nodes AiiDA generates empty directories in the repository.
In my case I removed some of these to reduce the number of inodes I have. The database export and import now poses a problem since it runs into these (false positive) exception:
aiida-core/aiida/tools/importexport/dbexport/__init__.py
Line 500 in 12f9641
and
aiida-core/aiida/tools/importexport/dbimport/backends/django/__init__.py
Line 392 in 12f9641
It would probably be good to not require empty directories to be included in import/export files.
The text was updated successfully, but these errors were encountered: