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

BUG: WAVECAR deletion #769

Open
JaGeo opened this issue Mar 9, 2024 · 8 comments
Open

BUG: WAVECAR deletion #769

JaGeo opened this issue Mar 9, 2024 · 8 comments

Comments

@JaGeo
Copy link
Member

JaGeo commented Mar 9, 2024

Describe the bug
In my current tests of the MPVaspLobsterMaker, the WAVECARs are not correctly deleted. It's a purely jobflow-based run. We would need to investigate this in detail.

@naik-aakash have you seen issues like this?

@naik-aakash
Copy link
Contributor

Umm, I did not see this. But I can check it 😃

@JaGeo JaGeo changed the title BUG: WAVECAR deleltion BUG: WAVECAR deletion Mar 10, 2024
@JaGeo
Copy link
Member Author

JaGeo commented Mar 10, 2024

I ran the workflow with jobflow on supermuc.

Could you also try running the workflow with jobflow remote? I get weird key errors when the lobster job is saved. It has nothing to do with size but with invalid keys.

@naik-aakash
Copy link
Contributor

naik-aakash commented Mar 11, 2024

Is it related specifically to the Bandoverlaps data key or any specific field? And any specific compound?

@JaGeo
Copy link
Member Author

JaGeo commented Mar 11, 2024

Mg3Sb2. The rest is not clear as the outputs are not saved. It would need further testing

@naik-aakash
Copy link
Contributor

Thanks will look into it. 😄

@naik-aakash
Copy link
Contributor

Hi @JaGeo ,Checked it with jobflow remote and it seems to originate from band_overlaps field. I will test with the recent version of our Schema (open PR), I think that should have fixed this issue already as we do not store it as dict anymore but as data object.

Seems earlier pydantic version did not have such checks implemented ,so never encountered this issue before.

@JaGeo
Copy link
Member Author

JaGeo commented Mar 11, 2024

I tried it with the recent version as well (also your branch). It also failed for me

@naik-aakash
Copy link
Contributor

Yeah , I just finished checking the same. Seems I need to update keys of bandoverlaps object in pymatgen to make it work with pydantic checks...

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