Skip to content

Stabilize thermochemistry workflow - #24

Merged
galjos merged 1 commit into
mainfrom
stabilize-thermo-workflow
Jun 22, 2026
Merged

Stabilize thermochemistry workflow#24
galjos merged 1 commit into
mainfrom
stabilize-thermo-workflow

Conversation

@galjos

@galjos galjos commented Jun 22, 2026

Copy link
Copy Markdown
Member

Summary

  • Fix source-tree version lookup and package entry point.
  • Implement .gen coordinate parsing and harden input parsing.
  • Improve DFTB+ path handling and executable checks.
  • Trim unused dependencies and add focused regressions.

Validation

  • python -m pytest --cov=ThermoScreening --cov-report=term -q -> 40 passed
  • DFTB+ integration tests -> included in the full run
  • Clean Linux container with DFTB+ and coverage -> 40 passed

@codecov

codecov Bot commented Jun 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.01%. Comparing base (b21764a) to head (41eefd1).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
+ Coverage   82.47%   87.01%   +4.54%     
==========================================
  Files          19       20       +1     
  Lines        1044     1063      +19     
==========================================
+ Hits          861      925      +64     
+ Misses        183      138      -45     
Flag Coverage Δ
unittests 87.01% <100.00%> (+4.54%) ⬆️

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

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@galjos
galjos force-pushed the stabilize-thermo-workflow branch from 854f204 to 99cd079 Compare June 22, 2026 10:12
@galjos
galjos force-pushed the stabilize-thermo-workflow branch from 99cd079 to 41eefd1 Compare June 22, 2026 10:35
@galjos
galjos marked this pull request as ready for review June 22, 2026 10:54
@galjos
galjos merged commit d90b781 into main Jun 22, 2026
3 checks passed
@galjos
galjos deleted the stabilize-thermo-workflow branch June 22, 2026 11:01
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