-
Notifications
You must be signed in to change notification settings - Fork 49
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
WIP: Added ase Nudged Elastic Band in the NewtonNet recipe #2176
Open
kumaranu
wants to merge
462
commits into
Quantum-Accelerators:main
Choose a base branch
from
kumaranu:neb_nn
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 3 commits
Commits
Show all changes
462 commits
Select commit
Hold shift + click to select a range
6f92356
Auto stash before merge of "neb_nn" and "origin/neb_nn"
kumaranu 0314718
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu a4f684f
Auto stash before merge of "neb_nn" and "origin/neb_nn"
kumaranu e7c0311
Added test for BFGSLinesearch exception.
kumaranu a31f20c
pre-commit auto-fixes
pre-commit-ci[bot] 60050fb
added docstring to test_run_neb2
kumaranu c18f48b
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 027e759
pre-commit auto-fixes
pre-commit-ci[bot] 0840c8c
Auto stash before merge of "neb_nn" and "origin/neb_nn"
kumaranu c1427d8
Removed unused code from test_run_neb
kumaranu 1646d42
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu bfc9ced
pre-commit auto-fixes
pre-commit-ci[bot] 716e628
Auto stash before merge of "neb_nn" and "origin/neb_nn"
kumaranu a9d5ff2
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 30d11dc
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 33b8d4e
pre-commit auto-fixes
pre-commit-ci[bot] f615130
Added docstrings to summarize_neb_run and summarize_neb_run2
kumaranu 40c29ca
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 87a08ce
Got rid of random molecule and replaced it with C60.
kumaranu dc10ecc
pre-commit auto-fixes
pre-commit-ci[bot] 4fea200
Added seed to test_ase for geodesic.
kumaranu 6c6583b
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu f18ea76
pre-commit auto-fixes
pre-commit-ci[bot] f06ac43
Merge branch 'main' into neb_nn
Andrew-S-Rosen 28ced33
pre-commit auto-fixes
pre-commit-ci[bot] b13b4aa
Added print statements for newtonnet model and config.
kumaranu 953ec3a
pre-commit auto-fixes
pre-commit-ci[bot] b3c9c1a
Auto stash before merge of "neb_nn" and "origin/neb_nn"
kumaranu 04e92dc
pre-commit auto-fixes
pre-commit-ci[bot] 411ac3c
Trying noqa to get rid of pre-commit warning for random seed.
kumaranu 2acf47f
Moved optimizer import.
kumaranu 695febf
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu f44380d
Merge branch 'main' into neb_nn
kumaranu dc124cc
Auto stash before merge of "neb_nn" and "origin/neb_nn"
kumaranu d6f8b5f
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 62ab50a
pre-commit auto-fixes
pre-commit-ci[bot] fe98bce
Auto stash before merge of "neb_nn" and "origin/neb_nn"
kumaranu 92b10d3
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 9e76d64
Auto stash before merge of "neb_nn" and "origin/neb_nn"
kumaranu 1cadf16
pre-commit auto-fixes
pre-commit-ci[bot] f9c854b
Just trying to run the tests again.
kumaranu ad207b8
Added strip decorators to the ts_job calls inside neb_ts_job and geod…
kumaranu 109c1d8
Added hess_method=None to avoid doing hessian calculations all the time.
kumaranu 4143de1
Merge branch 'main' into neb_nn
kumaranu c79b2a1
Added opt_kwargs to the ts_job calls to allow hessian=True sella calc…
kumaranu 2774be2
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu c85d539
pre-commit auto-fixes
pre-commit-ci[bot] 70b43d3
Made tests more robust for newtonnet.
kumaranu bf0a280
Correct calc_kwargs and opt_kwargs to make sure hessians are calculat…
kumaranu c24cb76
Modified add hessian and std function to allow calc_kwargs.
kumaranu adc8396
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu a1c3c9f
pre-commit auto-fixes
pre-commit-ci[bot] 09c9a4d
Changed ts_job call according to the requirements.
kumaranu dcd284a
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 01105ab
just corrected the newtonnet tests.
kumaranu 0a0cff4
pre-commit auto-fixes
pre-commit-ci[bot] 5400196
Changed opt_kwargs to ts_job_kwargs
kumaranu 9575103
Updated newtonnet tests for ts_job_kwargs.
kumaranu 9697a29
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 8c6b015
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 160fa21
Merge branch 'main' into neb_nn
Andrew-S-Rosen ca46eae
Merge remote-tracking branch 'origin/main' into neb_nn
kumaranu 42d0da2
Merge branch 'Quantum-Accelerators:main' into neb_nn
kumaranu a7cdef2
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 91d556a
Auto stash before merge of "neb_nn" and "origin/neb_nn"
kumaranu c025d0d
Added additional fields to neb_summary.
kumaranu 03e23a2
edited tests too. May be will change them later.
kumaranu 045facb
pre-commit auto-fixes
pre-commit-ci[bot] 191e3a8
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 9ff5d75
added geodesic_job the schemas are not added yet.
kumaranu b8d3f6a
added test for geodesic job
kumaranu d19ce44
pre-commit auto-fixes
pre-commit-ci[bot] fe3f2cf
Merge branch 'main' into neb_nn
kumaranu 4254141
modified the schemas for neb_job to only contain n_images of the traj…
kumaranu ea6392f
Test updated for neb job to check for the length for neb_job.
kumaranu 90943e0
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu fa830b1
pre-commit auto-fixes
pre-commit-ci[bot] 363bd0c
Corrected the tests.
kumaranu ec3bfd8
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 20c3443
Merge branch 'main' into neb_nn
kumaranu 9bb1c39
Corrected TSSchema and OptSchema in ts.py.
kumaranu 2f5b531
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu f2ef51b
corrected OptSchema imports.
kumaranu f8b7438
Correct run_neb test.
kumaranu f88a6ba
pre-commit auto-fixes
pre-commit-ci[bot] 4253d94
Edited runners/test_ase.py to pass the tests.
kumaranu df6c245
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu a0245c5
Added check environment to look for environment on quacc GitHub.
kumaranu 273ba6b
Changed seed setting from session to module.
kumaranu 741c780
Changed number according to GitHub tests on github.com.
kumaranu 54312a7
Corrected numbers in the test_run_neb.
kumaranu 994dd46
Corrected the value in pytest for run_neb.
kumaranu 93e1356
Merge branch 'main' into neb_nn
kumaranu 6533f83
Changed the scope of the random seed to module from session. It was s…
kumaranu d709cec
Added geodesic_path.xyz file to directly load the gedesic path for th…
kumaranu 4c1b3a8
Removed unnecessary path of the code after replacing the geodesic pat…
kumaranu 72b0134
pre-commit auto-fixes
pre-commit-ci[bot] 1838b1c
Added n returns to neb test.
kumaranu 13cdff4
Got rid of summar_neb_run2.
kumaranu 59fc5e3
Added n_iter_return = -1 by default inside neb_job.
kumaranu b1ea1d2
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu b831ab8
pre-commit auto-fixes
pre-commit-ci[bot] 35a13ae
Removed docstring from a test file.
kumaranu 502253b
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 44d365a
Removed geodesic from tests/requirements.txt.
kumaranu 7a8924b
Corrected the relative geodesic path for geodesic_path.xyz file in th…
kumaranu b870a7d
Added geodesic as a requirement inside the github actions job require…
kumaranu 59ace9c
pre-commit auto-fixes
pre-commit-ci[bot] 160b232
Added trajectory import.
kumaranu acce40e
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu c0c9655
pre-commit auto-fixes
pre-commit-ci[bot] 8ae1bde
Reduced the relative threshold for error check for neb.
kumaranu 18fd593
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 49da3eb
Changed a few numbers here to match the tests on GitHub.
kumaranu 3120fae
swapped neb_kwargs with run_kwargs. Suggested by Andrew.
kumaranu dae5796
Added a dynamics object in the returned type hint.
kumaranu d2d8ffd
Update some loop related to periodic systems.
kumaranu 9a6c9ca
Removed a stray comment.
kumaranu 5ce7cd8
Cleaned up the traj file related things from run_neb.
kumaranu c38b6c8
pre-commit auto-fixes
pre-commit-ci[bot] 11cd0b9
Corrected a docstring.
kumaranu 905c906
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu cba79d3
Added sweep_cutoff_size and also change the "None" to "auto" in the g…
kumaranu 01bfb17
Corrected the geodesic test to incorporate new tol args.
kumaranu 979d193
Changed the geodesic test to directly put the variables inside the fu…
kumaranu 0ff073c
Added tests for linear and idpp interpolation tests. Also edited the …
kumaranu d3eab91
Corrected the schema neb function to deal with the interpolate_flags …
kumaranu dc7c80b
Added interpolation methods from ase like linear and idpp in addition…
kumaranu bb58d2e
Removed has_geodesic_interpolate condition from neb_job.
kumaranu 99bc8d5
pre-commit auto-fixes
pre-commit-ci[bot] 39494c0
Changed geodesic_interpolate_flags to interpolate_flags inside runner…
kumaranu 3468dcb
Increased the accuracy condition from 1e-2 to 1e-5 for geodesic neb t…
kumaranu 6ba14ef
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 45c4070
Update test numbers in newtonnet tests.
kumaranu a12fad8
Corrected neb_ts_job function to have correct arguments getting passe…
kumaranu dd1c6cb
pre-commit auto-fixes
pre-commit-ci[bot] 51aac67
Not sure why the geodesic numbers are not matching but updated them i…
kumaranu 5990dfc
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 9c47266
Added paths to kwargs.
kumaranu da6c3af
Corrected neb_kwargs text in docstrings
kumaranu 32ad251
Completely got rid of links for geodesic kwargs.
kumaranu 6eb3582
Changed calc_kwargs to **calc_kwargs.
kumaranu bf752c4
Also changed the test calls to use **calc_kwargs instead of calc_kwar…
kumaranu b2b0466
pre-commit auto-fixes
pre-commit-ci[bot] d028529
Merge branch 'main' into neb_nn
Andrew-S-Rosen 43d1504
Removed check_env.yml file.
kumaranu b9c8bd0
Modified _get_hessian to incorporate **calc_kwargs to allow used defi…
kumaranu 2c46a94
pre-commit auto-fixes
pre-commit-ci[bot] b7cf522
Trying the resolve the issue with opt.traj's path.
kumaranu 299c440
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 439de1e
Converted geodesic_path to string to have posix_path functions applie…
kumaranu 1bd71bf
pre-commit auto-fixes
pre-commit-ci[bot] 36d933e
Changed the posixpath to string in os.path.dirname(str(os.path.abspat…
kumaranu 314851d
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu fa02d79
changed the posixpath inside os.path.dirname function.
kumaranu 7db0b1f
pre-commit auto-fixes
pre-commit-ci[bot] d737a93
Merge branch 'main' into neb_nn
kumaranu 0a40aa8
Trying to convert posixpath to str at: traj.filename = str(zpath(…
kumaranu 81b05ce
Corrected the input with string inside zpath as suggested by Andrew.
kumaranu 7187de3
Undid the changes in the test file for the posixpath to string conver…
kumaranu 04bafa4
Moved custom type hints from ts.py to types.py
kumaranu 2a7645a
pre-commit auto-fixes
pre-commit-ci[bot] 6bea8a7
Imported the type-hints from types.py.
kumaranu 9356e8b
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu e5c3aee
pre-commit auto-fixes
pre-commit-ci[bot] c1704a5
Merge branch 'main' into neb_nn
kumaranu d5b481e
Removed a comment about 5 images in the neb_job function.
kumaranu 5689e2b
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu a673acd
Changed the key from geodesic_results to initial_images.
kumaranu afa79da
Again, changed the keys in the neb's returned schema from geodesic_re…
kumaranu c2605a5
pre-commit auto-fixes
pre-commit-ci[bot] c2d43f6
Changed a variable's name from i to result to make it more descriptive.
kumaranu 9377a82
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu f9e4af6
pre-commit auto-fixes
pre-commit-ci[bot] 326f925
Added typehints for geodesic jobs.
kumaranu 6fa67ee
updated ts.py to have geodesic schemas instead of dicts.
kumaranu b88c40c
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 4b33856
pre-commit auto-fixes
pre-commit-ci[bot] 58766ee
Removed an unnecessary check.
kumaranu ec95533
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 7adf08e
Moved geodesic_path.xyz file to a directory called test_files.
kumaranu 50eaf8b
Updated tests to have correct path to the geodesic_path.xyz file.
kumaranu e446487
Got rid of tmp_path or unused/empty calc_kwargs from newtonnet's test…
kumaranu e263622
pre-commit auto-fixes
pre-commit-ci[bot] 60b7aef
Removed neb_ts_job and geodesic_ts_job
kumaranu 93cf42d
Removed tests for neb_ts_job and geodesic_ts_job and the respective i…
kumaranu 003e056
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu ee28b48
pre-commit auto-fixes
pre-commit-ci[bot] a069d89
Removed tests for nen_ts_job and geodesic_ts_job.
kumaranu 4c2d20f
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 24b04cd
Manually edited the recipes list in the docs. Not sure if this was th…
kumaranu 5483d27
Not sure why this test failed. Trying the new value.
kumaranu 79ec33d
Added a test to raise a value error in run_neb function when optimize…
kumaranu 7da2d1d
Modified run_neb to have none as neb_kwargs.
kumaranu 0130aa3
pre-commit auto-fixes
pre-commit-ci[bot] 2ff1bef
Added Callable back in ase.py.
kumaranu 0d79cbf
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 7b9a9e4
pre-commit auto-fixes
pre-commit-ci[bot] 3c78709
Updated BFGSLineSearch check.
kumaranu 8cab2cc
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu a41fb04
Removed underscore from geodesic because it is not private.
kumaranu 0665ee6
Removed underscore from geodesic because it is not private.
kumaranu ad975ab
Removed underscore from geodesic because it is not private.
kumaranu e36c3ea
Changed tolerance variable names to redestribute_tol and smoother_tol.
kumaranu 5a8b6f0
Changed tolerance variable names to redestribute_tol and smoother_tol.
kumaranu 93a93b6
pre-commit auto-fixes
pre-commit-ci[bot] f6e789b
Changed the default number of n_images in geodesic to 10 instead of 20.
kumaranu f75f70a
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 00b0e63
corrected spelling for redistribute.
kumaranu 7994881
Corrected assertion value.
kumaranu bf05f39
Moved geodesic wrapper function from ts ase to atoms directory. Also …
kumaranu eac9687
Removed the geodesic tests from test_ase.py and the relevant imports.
kumaranu d363d5e
test file for the geodesic wrapper.
kumaranu b47725a
getting rid of the geodesic wrapper from ase.
kumaranu 25e79a0
pre-commit auto-fixes
pre-commit-ci[bot] d545f8b
Changed summarize_neb_run call.
kumaranu 30a7dc6
Made n_images require argument in summarize_neb_run and n_iter_return…
kumaranu 6023819
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 1a4b1a1
Updated run_neb test.
kumaranu dc4cd86
pre-commit auto-fixes
pre-commit-ci[bot] 4e8a05c
Merge branch 'main' into neb_nn
Andrew-S-Rosen db157db
Update ts.py
Andrew-S-Rosen 19685f1
Merge branch 'main' into neb_nn
kumaranu c2ef8cb
Removed calculator from the trajectory object in summarize neb run.
kumaranu f140801
Changed trajectory to atoms_trajectory in summarize_neb_run to avoid …
kumaranu 18a58dc
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 05862d9
Changed i to result defined for a loop over trajectory_results.
kumaranu 5465aa1
pre-commit auto-fixes
pre-commit-ci[bot] 2c867db
Added typehints and docstrings to the function inside the summarize f…
kumaranu 7c9e819
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu 4d9dcc8
pre-commit auto-fixes
pre-commit-ci[bot] 5d92bfa
edited docstring.
kumaranu c72b95a
Merge branch 'neb_nn' of https://github.com/kumaranu/quacc into neb_nn
kumaranu fe00971
pre-commit auto-fixes
pre-commit-ci[bot] 9a43dff
Merge branch 'main' into neb_nn
kumaranu c5ddcfa
Merge branch 'main' into neb_nn
Andrew-S-Rosen 7128288
Merge branch 'main' into neb_nn
Andrew-S-Rosen 777160d
Merge branch 'main' into neb_nn
Andrew-S-Rosen 68d7ec9
Update ase.py
Andrew-S-Rosen 2ba31ad
Update ase.py
Andrew-S-Rosen fce697e
Merge branch 'Quantum-Accelerators:main' into neb_nn
kumaranu e35f912
Removed the directory paths from the log and the json files for the o…
kumaranu 49a21db
pre-commit auto-fixes
pre-commit-ci[bot] fa3150f
Merge branch 'main' into neb_nn
Andrew-S-Rosen d65aef9
pre-commit auto-fixes
pre-commit-ci[bot] 7bbff3b
Merge branch 'main' into neb_nn
Andrew-S-Rosen 7c680a9
pre-commit auto-fixes
pre-commit-ci[bot] 0f99a89
Remove unused code block
Andrew-S-Rosen 6e374de
Merge branch 'main' into neb_nn
Andrew-S-Rosen c67c622
Merge branch 'main' into neb_nn
Andrew-S-Rosen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Andrew-S-Rosen marked this conversation as resolved.
Show resolved
Hide resolved
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can't have direct git links in
pyproject.toml
(unfortunately) because PyPI does not allow us to upload new version of quacc with URLs. The best course of action here is to simply make it clear in the@requires
decorator that this package should be installed.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @Andrew-S-Rosen ,
I want to have tests that run for geodesic interpolate on GitHub. Is there a way you see to do that in case we are not able to put URLs?
Thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kumaranu: Simply remove the direct link from
pyproject.toml
. Therequirements.txt
files in thetests
folder are what is used by CI.