Readd aims magnetism test

[tpurcell90]

Summary

The aims magnetism test should now work.

Checklist

Before a pull request can be merged, the following items must be checked:

• Code is in the standard Python style.
  The easiest way to handle this is to run the following in the correct sequence on
  your local machine. Start with running ruff and ruff format on your new code. This will
  automatically reformat your code to PEP8 conventions and fix many linting issues.
• Doc strings have been added in the Numpy docstring format.
  Run ruff on your code.
• Type annotations are highly encouraged. Run mypy to
  type check your code.
• Tests have been added for any new functionality or bug fixes.
• All linting and tests pass.

Note that the CI system will run all the above checks. But it will be much more
efficient if you already fix most errors prior to submitting the PR. It is highly
recommended that you use the pre-commit hook provided in the repository. Simply run
pre-commit install and a check will be run prior to allowing commits.

[tpurcell90]

This can merge in once the pymatgen update is possible

[JaGeo]

Yeah. Should hopefully be possible soon. OpenMM tests behave weird again. Will try to fix them and then see....

[tpurcell90]

No rush, I was just checking to see if that was the fail point on here since the test pass locally.

[esoteric-ephemera]

@tpurcell90 I'm seeing that some of the magnetic moments are None when running the AIMS magnetism test - any ideas if this is an issue on pymatgen's side for the AIMS input set?

[tpurcell90]

Let me check what is going on here later today

[tpurcell90]

This looks to be on the pymatgen side. I will fix this once: materialsproject/pymatgen#4260 gets merged in

[esoteric-ephemera]

Great thanks!

[tpurcell90]

This should now work if the latest pymatgen and pymatgen-io-aims are installed. Should I add the new aims extra to strict as well?

[esoteric-ephemera]

Good question - given the precedent with abinit and amset, I don't think you need to put them in strict unless there's a conflict with the most recent version of another package like pymatgen

[tpurcell90]

At this point I think it works with the master version of pymatgen with the aims plug in locally, so I can confirm once that is pip accessible

[esoteric-ephemera]

Great thanks! I'll bump the pymatgen dependence once it's released and merge if all the tests pass