Skip to content

perf: Improving performance of adding memberships from records#104

Merged
pesap merged 9 commits intomainfrom
pesap/benchmark
Mar 15, 2026
Merged

perf: Improving performance of adding memberships from records#104
pesap merged 9 commits intomainfrom
pesap/benchmark

Conversation

@pesap
Copy link
Collaborator

@pesap pesap commented Mar 5, 2026

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Mar 5, 2026

Codecov Report

❌ Patch coverage is 98.38710% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 95.63%. Comparing base (0e572a0) to head (70c0738).

Files with missing lines Patch % Lines
src/plexosdb/db.py 97.56% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #104      +/-   ##
==========================================
+ Coverage   95.20%   95.63%   +0.43%     
==========================================
  Files           8        8              
  Lines        1688     1719      +31     
==========================================
+ Hits         1607     1644      +37     
+ Misses         81       75       -6     

☔ View full report in Codecov by Sentry.
📢 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.

- Upgrade actions/cache from v4 to v5 (node24) to fix Node.js 20 deprecation warning
- Initialize benchmark-data.json before publish step to prevent ENOENT warning on first run
- Use github.head_ref || github.ref_name so push and PR runs share the same branch cache
- Add main branch fallback in restore-keys so new branches get baseline comparison data
@pesap pesap merged commit 1ea4a39 into main Mar 15, 2026
26 checks passed
@pesap pesap deleted the pesap/benchmark branch March 15, 2026 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants