Skip to content

minor change to avoid warning from ase #22

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

yury-lysogorskiy
Copy link
Member

on my system (fresh install, ase 3.22.1 ) the relevant part of the traceback:

    File "/scratch/AnacondaEnvironments/ml_datasets/lib/python3.9/site-packages/ase/atoms.py", line 731, in get_potential_energy
      energy = self._calc.get_potential_energy(self)
    File "/scratch/AnacondaEnvironments/ml_datasets/lib/python3.9/site-packages/ase/calculators/calculator.py", line 709, in get_potential_energy
      energy = self.get_property('energy', atoms)
    File "/scratch/AnacondaEnvironments/ml_datasets/lib/python3.9/site-packages/ase/calculators/calculator.py", line 737, in get_property
      self.calculate(atoms, [name], system_changes)
    File "/scratch/AnacondaEnvironments/ml_datasets/lib/python3.9/site-packages/pyace/asecalc.py", line 161, in calculate
      if self.atoms.number_of_lattice_vectors == 3:
    File "/scratch/AnacondaEnvironments/ml_datasets/lib/python3.9/site-packages/ase/utils/__init__.py", line 62, in deprecated_function
      warnings.warn(warning)
  FutureWarning: Please use atoms.cell.rank instead

which is solved with this modification

on my system (fresh install, ase 3.22.1 )  the relevant part of the
traceback:

```
    File "/scratch/AnacondaEnvironments/ml_datasets/lib/python3.9/site-packages/ase/atoms.py", line 731, in get_potential_energy
      energy = self._calc.get_potential_energy(self)
    File "/scratch/AnacondaEnvironments/ml_datasets/lib/python3.9/site-packages/ase/calculators/calculator.py", line 709, in get_potential_energy
      energy = self.get_property('energy', atoms)
    File "/scratch/AnacondaEnvironments/ml_datasets/lib/python3.9/site-packages/ase/calculators/calculator.py", line 737, in get_property
      self.calculate(atoms, [name], system_changes)
    File "/scratch/AnacondaEnvironments/ml_datasets/lib/python3.9/site-packages/pyace/asecalc.py", line 161, in calculate
      if self.atoms.number_of_lattice_vectors == 3:
    File "/scratch/AnacondaEnvironments/ml_datasets/lib/python3.9/site-packages/ase/utils/__init__.py", line 62, in deprecated_function
      warnings.warn(warning)
  FutureWarning: Please use atoms.cell.rank instead
```

which is solved with this modification
@mdforti mdforti force-pushed the master branch 2 times, most recently from 165dca3 to f2bf2ca Compare December 17, 2022 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants