Skip to content

fix(database/csv): Improve CSV file I/O reliability - #897

Merged
gfieni merged 12 commits into
masterfrom
fix/csv-fileio-handlers
Jul 31, 2026
Merged

fix(database/csv): Improve CSV file I/O reliability#897
gfieni merged 12 commits into
masterfrom
fix/csv-fileio-handlers

Conversation

@gfieni

@gfieni gfieni commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

This PR improves the correctness, reliability, and performance of the CSV database implementation.

Changes overview:

  • Improve the reliability of CSV read and write operations.
  • Strengthen resource handling and invalid-input detection.
  • Standardize error reporting across the CSV driver.
  • Optimize HWPCReport decoding.
  • Reorganize and expand CSV unit test coverage.

@gfieni gfieni self-assigned this Jul 31, 2026
@gfieni
gfieni requested a review from a team as a code owner July 31, 2026 13:11
@gfieni gfieni added the enhancement New feature or request label Jul 31, 2026
@codecov

codecov Bot commented Jul 31, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 81.25000% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.29%. Comparing base (619e7e6) to head (dd261e6).
⚠️ Report is 13 commits behind head on master.

Files with missing lines Patch % Lines
src/powerapi/database/csv/driver.py 25.00% 9 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #897      +/-   ##
==========================================
+ Coverage   73.92%   77.29%   +3.36%     
==========================================
  Files          83       83              
  Lines        3129     3144      +15     
==========================================
+ Hits         2313     2430     +117     
+ Misses        816      714     -102     
Files with missing lines Coverage Δ
src/powerapi/database/csv/codecs.py 94.11% <100.00%> (ø)
src/powerapi/database/csv/fileio_handlers.py 100.00% <100.00%> (+71.90%) ⬆️
src/powerapi/database/csv/driver.py 69.11% <25.00%> (+20.54%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gfieni
gfieni force-pushed the fix/csv-fileio-handlers branch from e73b21f to dd261e6 Compare July 31, 2026 13:46
@sonarqubecloud

Copy link
Copy Markdown

@gfieni
gfieni merged commit e921d93 into master Jul 31, 2026
10 of 11 checks passed
@gfieni
gfieni deleted the fix/csv-fileio-handlers branch July 31, 2026 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant