override_default_vasp_params cannot be supplied as an input for the argument optimizefw_kwargs in get_ion_insertion_wf (https://github.com/hackingmaterials/atomate/blob/cf04d75ab75ffc565cc160df96c81de751eee4b1/atomate/vasp/workflows/base/electrode.py#L32) because of how the GetInsertionCalcs firetask is written. As it currently stands, the Charge Density Analysis firework will fizzle with the following error:
TypeError: ABCMeta object got multiple values for keyword argument ‘override_default_vasp_params’
I would suggest removing automatically setting NSW=299 because this may conflict with user preferences and is inconsistent with the first structure optimization firework in the workflow. I can make a pull request on github with this change but would welcome other thoughts on how to best resolve this bug.