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

Encounternig warning trying to import "aiida_tutorial_2019_05_perovskites_v0.3.aiida" #184

Closed
csadorf opened this issue Oct 4, 2019 · 7 comments
Assignees

Comments

@csadorf
Copy link
Collaborator

csadorf commented Oct 4, 2019

Trying to execute the verdi import https://object.cscs.ch/v1/AUTH_b1d80408b3d340db9f03d373bbde5c1e/marvel-vms/tutorials/aiida_tutorial_2019_05_perovskites_v0.3.aiida, I get the following warning:

Warning: Export file version is 0.4, can import only version 0.7

Does this need to be addressed?

@sphuber
Copy link
Contributor

sphuber commented Oct 7, 2019

We decided to keep this, because all the user has to do is press enter for the archive to be automatically migrated before being imported. But @CasperWA and @yakutovicha if you prefer it fixed we will have to migrate the archives ourselves and upload the new version, while updating the permanent link in the tutorial text

@csadorf
Copy link
Collaborator Author

csadorf commented Oct 7, 2019

Alternatively the warning could be addressed in the tutorial or we don't do anything. I just reported this, because this warning and the additionally required user action represent a slight mismatch between what is presented in the tutorial and the UX.

@ltalirz
Copy link
Member

ltalirz commented Jun 30, 2020

This issue persists and it would be nice to upload the migrated file.

@sphuber
Copy link
Contributor

sphuber commented Jun 30, 2020

I have migrated the file. Who can upload this to the Materials Cloud?
File is here: https://drive.google.com/file/d/1FWuOrgsuBDtDt6FLXegB4IEdNGUwYZR0/view?usp=sharing
Archive now has v0.9 which is compatible with aiida-core==1.3.0

@ltalirz
Copy link
Member

ltalirz commented Jun 30, 2020

Thanks, will do!

@csadorf
Copy link
Collaborator Author

csadorf commented Jul 1, 2020

@ltalirz Is this issue resolved with the merge of #236 ?

@ltalirz
Copy link
Member

ltalirz commented Jul 1, 2020

yes. I guess it would be closed automatically when the intro branch is merged in to develop.

@ltalirz ltalirz closed this as completed Jul 1, 2020
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

4 participants