Skip to content

ASE calculator requires system_changes in call to calculate() #2

@bernstei

Description

@bernstei

I'm having a small problem with the ASE interface. For various reasons, my wrapper calls calc.calculate(atoms, properties=properties) instead of atoms.calc = calc; E = atoms.get_potential_energy(). With most calculators that works, but with the ACE ASE calculator I get calculate() missing 1 required positional argument: 'system_changes'. In other calculators I've checked, e.g. quippy.potential.Potential, it's implemented as default None https://github.com/libAtoms/QUIP/blob/91794ea6dd9f75ff5a7390039694f048fceb07cf/quippy/quippy/potential.py#L164

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions