Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #428 +/- ##
=======================================
Coverage 87.26% 87.27%
=======================================
Files 52 52
Lines 6457 6461 +4
Branches 715 717 +2
=======================================
+ Hits 5635 5639 +4
Misses 803 803
Partials 19 19
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR aims to speed up SQLite persistence during simulations by tuning SQLite settings and reducing overhead in periodic batch inserts, and it updates the PyPI/TestPyPI publish workflow to handle existing versions more explicitly.
Changes:
- Tune SQLite PRAGMAs for higher sustained write throughput when
saveData()is configured. - Consolidate periodic writes into a single transaction and avoid repeated per-row value recomputation during inserts.
- Update PyPI/TestPyPI publish policy: fail on existing versions for PyPI pushes, and delete/re-publish for TestPyPI PRs; bump patch version.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/dsf/mobility/FirstOrderDynamics.cpp | Adds SQLite PRAGMA tuning; batches inserts under a shared transaction; refactors avg_stats computations/binds. |
| src/dsf/dsf.hpp | Bumps DSF patch version (5.3.0 → 5.3.1). |
| .github/workflows/pypi.yml | Adds version existence output, enforces “no overwrite” on PyPI pushes, and deletes existing TestPyPI versions on PRs before upload. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.