This might be a silly question, but is changing the "type" parameter in the json file possible? The following evaluates to typing.Union

type(eval('bool | str'))

Example:

>>> isinstance(True, eval('bool | str'))
True
>>> isinstance("string thing", eval('bool | str'))
True
>>> isinstance(55, eval('bool | str'))
False

That's actually an ingenious solution IMO, now that using of eval is not risky in this scenario.

However the only pitfall being that the alias of Union[x, y] as x | y is introduced in Python 3.10, and as the min supported version for pymatgen is 3.9 at this moment, we still cannot implement that change. But still I think it's a cleaner solution than mine, perhaps we could import Union for now until we move to Python 3.10+.

@janosh Can you please review this? Thanks. I believe the CI failing is unrelated to this PR.

I believe the CI failing is unrelated to this PR.

i think the issue is due to installing the latest torch in test.yml while matGL requires torch=2.2.1 or older

can you try pinning to torch 2.2.1 in CI with a TODO to remove pin later?

Yes sure. Let me have a try.

Looks like matgl has removed the torch version limitation, but still we need to wait for a new release. materialyzeai/matgl@d0bf202

afaik dgl isn't compatible with torch==2.3 yet, let alone 2.4 so don't think the matgl up pin should have been removed. i was advocating for a down pin in another thread. either way, let's revert the temp torch==2.2.1 pin here and just skip the matGL tests for now to keep things moving

Sure! Thanks for the input, and I know nothing about matgl :)

Thanks for reviewing!