Fix AttributeError NoneType has no attribute 'style'

[janosh]

When running VASP with INCAR tag kspacing instead of a KPOINTS file and encountering brmix 2 or 3 times. Reported by @esoteric-ephemera in atomate2 r2SCAN workflow.

[janosh]

@arosen93 Would be great if you could take a quick look if this seems sensible.

[Andrew-S-Rosen]

@janosh: Thanks for the ping. This seems sensible to me.

Are we certain that we can do if vi["KPOINTS"] is not None? In other words, we don't need to do the safer option of if vi.get("KPOINTS")?

[janosh]

Yeah, looks like the key is always set, just initialized to None is KPOINTS file not found.

But we don't really need if vi["KPOINTS"] is not None, could do if vi["KPOINTS"].

[codecov-commenter]

Codecov Report

Patch coverage: 90.90% and project coverage change: +0.01% 🎉

Comparison is base (1497cd4) 66.02% compared to head (a4e6cca) 66.04%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #283      +/-   ##
==========================================
+ Coverage   66.02%   66.04%   +0.01%     
==========================================
  Files          51       51              
  Lines        5549     5549              
==========================================
+ Hits         3664     3665       +1     
+ Misses       1885     1884       -1     

Files Changed	Coverage Δ
custodian/cli/run_vasp.py	0.00% <ø> (ø)
custodian/vasp/handlers.py	78.89% <90.90%> (-0.08%)	⬇️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.