update_user_incar_settings does not correctly set MAGMOMS for VASP jobs

[matthewkuner]

atomate2/src/atomate2/vasp/powerups.py

Line 14 in 0433cd8

def update_user_incar_settings(

If a previous structure has magmoms, those magmoms will be used with a higher priority than the magmoms set via the powerup update_user_incar_settings. I believe this may stem from this code snippet , but I could be mistaken.

It is also worth noting that Custodian, by default, has an option to not reuse the magmoms from a previous INCAR (which is the default)