Skip to content

Provision CI from conda-forge#194

Merged
awvwgk merged 5 commits intodftd4:mainfrom
awvwgk:ci
Feb 22, 2023
Merged

Provision CI from conda-forge#194
awvwgk merged 5 commits intodftd4:mainfrom
awvwgk:ci

Conversation

@awvwgk
Copy link
Copy Markdown
Member

@awvwgk awvwgk commented Feb 22, 2023

Using conda environments to provision the dependencies for CI testing seems more stable than using the base images.

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 22, 2023

Codecov Report

Merging #194 (7392aad) into main (488e35b) will decrease coverage by 17.16%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #194       +/-   ##
===========================================
- Coverage   59.60%   42.44%   -17.16%     
===========================================
  Files          46       31       -15     
  Lines        3307     2554      -753     
  Branches     1057     1047       -10     
===========================================
- Hits         1971     1084      -887     
- Misses        725      957      +232     
+ Partials      611      513       -98     
Impacted Files Coverage Δ
src/dftd4/utils.f90 0.00% <0.00%> (-50.00%) ⬇️
src/dftd4/output.f90 0.00% <0.00%> (-50.00%) ⬇️
app/driver.f90 0.00% <0.00%> (-41.18%) ⬇️
app/cli.f90 42.85% <0.00%> (-14.29%) ⬇️
app/main.f90 30.76% <0.00%> (-7.70%) ⬇️
src/dftd4/model.f90 74.80% <0.00%> (-4.73%) ⬇️
python/dftd4/test_pyscf.py
python/dftd4/interface.py
python/dftd4/test_interface.py
python/dftd4/qcschema.py
... and 12 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@awvwgk awvwgk changed the title Provision CI with conda-forge Provision CI from conda-forge Feb 22, 2023
@awvwgk awvwgk force-pushed the ci branch 2 times, most recently from 45d9a59 to 13fef78 Compare February 22, 2023 19:55
@marvinfriede
Copy link
Copy Markdown
Member

I also fixed the macos build (see here).

@awvwgk
Copy link
Copy Markdown
Member Author

awvwgk commented Feb 22, 2023

The CI breakage over time is quite annoying, I found that repairing it in dftd3/simple-dftd3 was far easier than here. Therefore, I copied over the CI from there and adapted it to match the additional requirements (BLAS/Lapack). Feel free to decide which variant you prefer.

@awvwgk awvwgk merged commit ee8b54f into dftd4:main Feb 22, 2023
@awvwgk awvwgk deleted the ci branch February 22, 2023 21:27
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.

2 participants