Skip to content

allow latin1 encoding for csv files #1203

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 20 commits into from
May 12, 2025
Merged

allow latin1 encoding for csv files #1203

merged 20 commits into from
May 12, 2025

Conversation

amaloney
Copy link
Collaborator

@amaloney amaloney commented May 2, 2025

No description provided.

@amaloney amaloney requested a review from ahuang11 May 2, 2025 22:14
@amaloney amaloney self-assigned this May 2, 2025
@amaloney
Copy link
Collaborator Author

amaloney commented May 2, 2025

I need to add tests and refactor, but this does allow latin1 encoded CSV files to be read without errors.

Copy link

codecov bot commented May 2, 2025

Codecov Report

Attention: Patch coverage is 76.36364% with 13 lines in your changes missing coverage. Please review.

Project coverage is 55.93%. Comparing base (c5df684) to head (a324a13).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
lumen/util.py 65.38% 9 Missing ⚠️
lumen/ai/controls.py 33.33% 2 Missing ⚠️
lumen/ai/ui.py 33.33% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1203      +/-   ##
==========================================
+ Coverage   55.87%   55.93%   +0.06%     
==========================================
  Files         113      113              
  Lines       17150    17201      +51     
==========================================
+ Hits         9582     9622      +40     
- Misses       7568     7579      +11     

☔ 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.

@amaloney amaloney marked this pull request as ready for review May 9, 2025 20:01
@ahuang11 ahuang11 enabled auto-merge (squash) May 12, 2025 18:56
@ahuang11 ahuang11 merged commit 6ca7551 into main May 12, 2025
12 checks passed
@ahuang11 ahuang11 deleted the latin1-csv-duckdb branch May 12, 2025 19:01
ahuang11 added a commit that referenced this pull request Jun 5, 2025
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