Skip to content
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

V1.4.0 #281

Merged
merged 17 commits into from
Jul 25, 2023
Merged

V1.4.0 #281

merged 17 commits into from
Jul 25, 2023

Conversation

t-young31
Copy link
Member

@t-young31 t-young31 commented Jul 9, 2023

autodE v1.4.0 release 🚀


Checklist

  • Test pass
  • Benchmark has been run

t-young31 and others added 16 commits February 20, 2023 19:34
…ing (#202)

* make autode compatible with windows: replace multiprocessing with joblib and loky

* remove comment autode/utils.py

* check total memory in utils

* use loky for parallel processing

* loky context is used for better handling of parallelisation

* fixes for timeout wrapper in POSIX

* use class instances instead of class attributes for Config

* pass parent Config state into child processes

* keep default timeout wrapper, also add new wrapper

* separate parallelisation scheme for windows, restore default for linux/mac

* fix test_config_in_worker_proc for posix

* codecov on windows

* add tests for config copy

* change config implementation for updating in worker process

* add context manager for temporary config, minor fix in experimental timeout for windows

* add tests for context manager

* use bash for installing xtb in CI runner

* add tests for cleanup after timeout

* exclude setup.py from codecov

* make config picklable and easily readable

* add test and minor fixes

* move xtb install script to .github/scripts

* use temporary_config for test_utils.py

* update changelog and contributor list

* resolve conflicts with v1.4.0

Co-authored-by: Tom Young <39765193+t-young31@users.noreply.github.com>
* update docs for windows installation and temporary_config

* update docs for windows installation and temporary_config

* minor edit

Co-authored-by: Tom Young <39765193+t-young31@users.noreply.github.com>
* xTB tests should run on windows

* fix tests and simplify checking

* replace if with assert check

* Update tests/test_utils.py

---------

Co-authored-by: tom <tom.young.316@gmail.com>
* Removal of deprecated functions [skip actions]

* Add force constant [skip actions]

* NEB refactoring

* Fix units for div mul

* Update changelog

* PR suggestions

---------

Co-authored-by: Shoubhik Maiti <17470159+shoubhikraj@users.noreply.github.com>
* etemp and gfn options for xTB

* fix tests

* update changelog
* stub commit

* add qa step

* fix qa step to use cartesian step size

* improvements to coordinates

* damping updates

* TRIM step updates

* hessian update tests

* TRIM tests

* fix qa step to use cartesian step size

* minor updates

* unfinished updates

* minor updates

* updates to QA

* optimiser plot

* optimiser plotting added

* stub for flowchart update

* flowchart update

* trust update

* damping optimiser

* hessian update tests

* DIC flag to disallow unconverged IBT

* improvements to coordinates

* damping updates

* TRIM step updates

* hessian update tests

* TRIM tests

* tests for trimoptimiser

* tests for plotting

* unconverged ibt is allowed by default

* add to init

* reject high energy increase

* print ratio as well

* BFGS-SR1 scheme added

* pr updates

* pr updates #2

* rename

* bfgs-sr1-update tests

* cartesian optimiser

* reimplement damping

* minor fixes

* pr updates, test updates

* fixes for tests, change name of RFO trust argument

* revert RFO and change step size control

* test updates

* xyz trajectory writing

* polynomial interpolation: commit

* pr suggestions

* test update

* changes to TRM

* use hessian eigenbasis

* fix 0 eigenvalue error
* Add exception for NEB init with < 2 images
* Raise exception if prune returns no conformers
* Fix QChem not using max_core
* plotting and neb modifications

* refactor print_geometries

* fix tests, remove unused code

* put test files in zip

* update to DHS

* type checking fixes

* reuse hessian by update instead of recalculating

* use bfgs-sr1

* code for calculation restored

* _method_name update

* unfinished updates, refactor

* pr updates 3

* update to imagepair

* add tests

* simplify basebracket class and add test

* doc updates

* bracket method name update

* refactor plotting unit conv

* revert neb code changes

* eip doc updates

* put bracket package in setup.py

* image pair flush old hessians
* fix multiprocessing spawn test

* increase timeout
* Resolve conflicts with v1.4.0

* Start removal of dep functions [skip actions]

* Add force constant [skip actions]

* NEB refactoring

* Fix units for div mul

* Update changelog

* Simplify and add comment

* Remove more deprecated functions [skip actions]

* Fix tests

* Ignore type checking in tests + more tests

* More typing

* More typing and update lint workflow

* Add some tests

* Typing and benchmark script exclusion

* Add test

* Pin in requirements file and use mamba

---------

Co-authored-by: Shoubhik Maiti <17470159+shoubhikraj@users.noreply.github.com>
* rename coordinates property and mypy fix

* angle calculation change

* change default trust radius

* pr suggestions
@codecov
Copy link

codecov bot commented Jul 9, 2023

Codecov Report

Merging #281 (fa2b22b) into master (161818c) will increase coverage by 0.03%.
The diff coverage is 95.31%.

@@            Coverage Diff             @@
##           master     #281      +/-   ##
==========================================
+ Coverage   97.16%   97.19%   +0.03%     
==========================================
  Files         196      201       +5     
  Lines       20214    22368    +2154     
==========================================
+ Hits        19641    21741    +2100     
- Misses        573      627      +54     
Flag Coverage Δ
unittests 97.19% <95.31%> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
autode/transition_states/templates.py 93.85% <ø> (ø)
autode/wrappers/keywords/basis_sets.py 100.00% <ø> (ø)
tests/test_conf_gen.py 100.00% <ø> (ø)
tests/test_pes/test_rs.py 97.19% <ø> (ø)
autode/utils.py 81.92% <48.45%> (-14.13%) ⬇️
tests/test_utils.py 83.52% <61.38%> (-13.00%) ⬇️
tests/test_graphs.py 98.93% <66.66%> (-1.07%) ⬇️
autode/transition_states/transition_state.py 75.00% <70.58%> (+10.48%) ⬆️
autode/transition_states/base.py 73.07% <76.92%> (+1.07%) ⬆️
autode/opt/coordinates/internals.py 97.22% <83.33%> (-0.88%) ⬇️
... and 120 more

... and 3 files with indirect coverage changes

@t-young31 t-young31 marked this pull request as ready for review July 25, 2023 07:55
@t-young31 t-young31 merged commit 4d66675 into master Jul 25, 2023
19 of 20 checks passed
@t-young31 t-young31 deleted the v1.4.0 branch July 25, 2023 08:09
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.

None yet

2 participants