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

DMA, Local force constants, Marcus dimension #265

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

DMA, Local force constants, Marcus dimension #265

wants to merge 371 commits into from

Conversation

eljost
Copy link
Owner

@eljost eljost commented Aug 7, 2023

  • removed deprecated and tests_staging
  • several bugfixes

@eljost eljost changed the title Dev DMA, Local force constants, Marcus dimension Aug 7, 2023
scripts/gen_m2m.py Fixed Show fixed Hide fixed
scripts/gen_m2m.py Fixed Show fixed Hide fixed
pysisyphus/wavefunction/dma.py Fixed Show fixed Hide fixed
"max_cycles": 1,
},
)
opt = opt_result.opt

Check notice

Code scanning / CodeQL

Unused local variable Note test

Variable opt is not used.
@codecov
Copy link

codecov bot commented Aug 8, 2023

Codecov Report

Attention: Patch coverage is 34.37692% with 4276 lines in your changes missing coverage. Please review.

Project coverage is 27.80%. Comparing base (5950976) to head (eda0102).
Report is 75 commits behind head on master.

Current head eda0102 differs from pull request most recent head 8bd72d3

Please upload reports for the commit 8bd72d3 to get more accurate results.

❗ There is a different number of reports uploaded between BASE (5950976) and HEAD (eda0102). Click for more details.

HEAD has 53 uploads less than BASE | Flag | BASE (5950976) | HEAD (eda0102) | |------|------|------| |unittests|54|1|
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #265       +/-   ##
===========================================
- Coverage   61.58%   27.80%   -33.78%     
===========================================
  Files         282      319       +37     
  Lines       29472    73521    +44049     
===========================================
+ Hits        18149    20440     +2291     
- Misses      11323    53081    +41758     
Flag Coverage Δ
unittests 27.80% <34.37%> (-33.78%) ⬇️

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

Files Coverage Δ
pysisyphus/TablePrinter.py 100.00% <100.00%> (ø)
pysisyphus/__init__.py 100.00% <100.00%> (ø)
pysisyphus/calculators/AFIR.py 89.82% <100.00%> (ø)
pysisyphus/calculators/AnaPotBase.py 55.14% <100.00%> (-0.21%) ⬇️
pysisyphus/calculators/MOPAC.py 28.71% <ø> (ø)
pysisyphus/calculators/__init__.py 100.00% <100.00%> (ø)
pysisyphus/cos/NEB.py 68.13% <ø> (ø)
pysisyphus/drivers/afir.py 33.04% <100.00%> (-0.22%) ⬇️
pysisyphus/drivers/pka.py 100.00% <100.00%> (ø)
pysisyphus/helpers.py 72.44% <100.00%> (-3.25%) ⬇️
... and 102 more

... and 4 files with indirect coverage changes

@eljost eljost force-pushed the dev branch 2 times, most recently from 01215f3 to 494de1c Compare August 9, 2023 08:06
import warnings

try:
import ase

Check notice

Code scanning / CodeQL

Unused import Note

Import of 'ase' is not used.
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

CodeQL found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.

Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

Johannes Steinmetzer added 6 commits October 2, 2024 15:08
pysisyphus goes full quantum now!
using scans along Normal modes and using a sum-over-states
partition function w/ eigenvalues determined via the Numerov method
now results are also dumped while the scan runs
fix: relaxed_scan.trj contained unrelaxed geometries
fix: revive 'pysisplot --scan'. this command plots files that match the
pattern "*relaxed_scan.dat"
Johannes Steinmetzer added 23 commits October 2, 2024 23:24
Now, the H2O2 tunnel splitting from the paper can be reproduced.
using Gaussian Process Regression w/ sklearn and optuna for
hyperparameter optimization.
See "tests/test_hindered_rotor/test_gpr_torsion.py" for a simple
example.

The general idea is to calculate the torsional potential energy curve
on-the-fly at points, predicted by an acquisition function, based on a
Gaussian process regression.

When the potential energy curve is in place, nuclear wavefunctions and
associated eigenvalues are evaluated using a periodic 1d-Numerov
approach. The required mass is determined according to Pitzer/East/Radom
(see partfuncs.inertmom).
add: higher level single point calcs in hindered_rotor
add: ability to use splined potentials for testing
as the reference image was created without it. I added a flag to disable
graphivz, to allow a meaningful comparison
restricted trust_radius and added hessian_recalculation
to hopefully make them avaiable for nix
as scipy shifted the whole fragment's centroid into the origin, which
was bad. Actually, the fragment must be shifted in a way, so the anchor
atom is in the origin.
mod: set overachieve factor 3 in hindered_rotor opts
now convergence of partition function and predicted standard deviation
are checked.
add: ability to scan transition states in hindered_rotor with
appropriate opt_cls key
add: hessian information is stored w/ correct prefix
fix: calculation of force constant using periodic finite differences
used one point too much
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.

1 participant