nomad-coe / electronic-parsers / 24408203814

93%

develop: 93%

push

github

Fix VASP time calculation dimensionality error

Fixed pint.DimensionalityError when parsing time_calculation and
time_physical values in VASP parser. The issue had two root causes:

1. time_calculation was assigned as plain value without units, causing
   unit mismatches when adding to time_physical (pint cannot add
   seconds + dimensionless)
2. time_initial could be None when previous calculations didn't set
   time_physical, causing dimensionality errors in arithmetic

Changes:
- Add explicit units (ureg.s) when setting time_calculation values
- Store time quantity in variable before assignment to avoid issues
  with metainfo unit wrapping
- Add None checks for time_initial with fallback to 0*ureg.s
- Add None checks for time_physical before using in calculations

All 23 VASP parser tests pass including test_malformed_time_entries.

8 of 8 new or added lines in 1 file covered. (100.0%)

36124 of 38889 relevant lines covered (92.89%)

0.93 hits per line