Skip to content

Better VASP config handling - #576

Merged
Andrew-S-Rosen merged 2 commits into
mainfrom
configsettings
Jul 10, 2023
Merged

Better VASP config handling#576
Andrew-S-Rosen merged 2 commits into
mainfrom
configsettings

Conversation

@Andrew-S-Rosen

Copy link
Copy Markdown
Member

No description provided.

@codecov

codecov Bot commented Jul 10, 2023

Copy link
Copy Markdown

Codecov Report

Merging #576 (3d69fc8) into main (e220be6) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

❗ Current head 3d69fc8 differs from pull request most recent head a2e6834. Consider uploading reports for the commit a2e6834 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #576      +/-   ##
==========================================
- Coverage   98.54%   98.53%   -0.01%     
==========================================
  Files          33       33              
  Lines        1647     1641       -6     
==========================================
- Hits         1623     1617       -6     
  Misses         24       24              
Impacted Files Coverage Δ
src/quacc/calculators/vasp.py 100.00% <100.00%> (ø)
src/quacc/custodian/vasp.py 97.77% <100.00%> (+0.05%) ⬆️
src/quacc/settings.py 97.56% <100.00%> (ø)

@Andrew-S-Rosen
Andrew-S-Rosen enabled auto-merge (squash) July 10, 2023 18:51
@Andrew-S-Rosen
Andrew-S-Rosen merged commit aa99acd into main Jul 10, 2023
@Andrew-S-Rosen
Andrew-S-Rosen deleted the configsettings branch July 10, 2023 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant