Skip to content

add ase calculator - #3

Merged
LiuGaoyong merged 8 commits into
mainfrom
2-add-ase-calculator
Jul 26, 2025
Merged

add ase calculator#3
LiuGaoyong merged 8 commits into
mainfrom
2-add-ase-calculator

Conversation

@LiuGaoyong

Copy link
Copy Markdown
Owner

No description provided.

@LiuGaoyong LiuGaoyong linked an issue Jul 26, 2025 that may be closed by this pull request

@LiuGaoyong LiuGaoyong left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test result on mac:

======================= test session starts =======================
platform darwin -- Python 3.12.11, pytest-8.4.1, pluggy-1.6.0
rootdir: /Users/lgy/GitHub/ZZZ/PyGFNFF
configfile: pyproject.toml
collected 2 items                                                 

test_co_by_ase.py Energy: -9.188eV
Forces: 
 [[-0.        -0.        -2.4781312]
 [-0.        -0.         2.4781312]]
Positions: 
 [[ 0.        0.        0.493003]
 [ 0.        0.       -0.657337]]
      Step     Time          Energy          fmax
BFGS:    0 19:39:44       -9.188347        2.478131
BFGS:    1 19:39:44       -9.050233        7.166941
BFGS:    2 19:39:44       -9.215231        0.433918
BFGS:    3 19:39:44       -9.215990        0.070653
BFGS:    4 19:39:44       -9.216011        0.000889
Energy: -9.216eV
C=O Length: 1.129Å
.
test_coffeine.py SPE for coffeine: 2.09ms
.

======================== 2 passed in 0.38s ========================

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test passed on windows by CI

@LiuGaoyong
LiuGaoyong merged commit ea227a7 into main Jul 26, 2025
4 checks passed
@LiuGaoyong
LiuGaoyong deleted the 2-add-ase-calculator branch July 26, 2025 12:24
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.

Add ASE Calculator

1 participant