search

nomad-coe / electronic-parsers

Apache License 2.0
18 stars 7 forks source link

Update parser tests for new thermostat metainfo #167

Closed JFRudzinski closed 9 months ago

JFRudzinski commented 9 months ago

fix FHIaims, check other tests

JosePizarro3 commented 9 months ago

Hi @JFRudzinski, I see the test_fhiaimsparser.py is failing. This has a very easy fix, as thermostat_parameters is a list, so line 259 should point to the [0] element.

Or are you planning other changes?

JFRudzinski commented 9 months ago

@JosePizarro3 Yes exactly. Would you mind making the change? I was planning to do it during the week. I did not anticipate how busy I would be but I have not had a second to breath 😮‍💨

JosePizarro3 commented 9 months ago

Sure, I'll quick fix it in #166 .