8000 fix: polars schema inference issue by joseph-sentry · Pull Request #876 · codecov/worker · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on May 5, 2025. It is now read-only.

fix: polars schema inference issue #876

Merged
merged 1 commit into from
Nov 11, 2024
Merged

fix: polars schema inference issue #876

merged 1 commit into from
Nov 11, 2024

Conversation

joseph-sentry
Copy link
Contributor

polars is having trouble inferring the schema for flags, i'm guessing that it's because at the start of the data that we're reading from, there's a bunch of null values for the flags, so i'm manually making it so the schema expects a list of strings

@joseph-sentry
fix: polars schema inference issue
256f06e 
polars is having trouble inferring the schema for flags, i'm guessing
that it's because at the start of the data that we're reading from,
there's a bunch of null values for the flags, so i'm manually making it
so the schema expects a list of strings
@joseph-sentry joseph-sentry requested review from ajay-sentry and a team November 11, 2024 17:04
@codecov Codecov
Copy link
codecov bot commented Nov 11, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.91%. Comparing base (675b8ff) to head (256f06e).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #876   +/-   ##
=======================================
  Coverage   97.91%   97.91%           
=======================================
  Files         444      444           
  Lines       35399    35399           
=======================================
  Hits        34661    34661           
  Misses        738      738
Flag Coverage Δ
integration 41.87% <ø> (ø)
unit 90.72% <ø> (ø)

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

Components Coverage Δ
NonTestCode 95.89% <ø> (ø)
OutsideTasks 95.72% <ø> (ø)
Files with missing lines Coverage Δ
tasks/cache_test_rollups.py 94.87% <ø> (ø)

@codecov-notifications
Copy link
codecov-notifications bot commented Nov 11, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #876   +/-   ##
=======================================
  Coverage   97.91%   97.91%           
=======================================
  Files         444      444           
  Lines       35399    35399           
=======================================
  Hits        34661    34661           
  Misses        738      738
Flag Coverage Δ
integration 41.87% <ø> (ø)
unit 90.72% <ø> (ø)

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

Components Coverage Δ
NonTestCode 95.89% <ø> (ø)
OutsideTasks 95.72% <ø> (ø)
Files with missing lines Coverage Δ
tasks/cache_test_rollups.py 94.87% <ø> (ø)

@codecov-public-qa Codecov Public QA
Copy link
codecov-public-qa bot commented Nov 11, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.91%. Comparing base (675b8ff) to head (256f06e).

✅ All tests successful. No failed tests found.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #876   +/-   ##
=======================================
  Coverage   97.91%   97.91%           
=======================================
  Files         444      444           
  Lines       35399    35399           
=======================================
  Hits        34661    34661           
  Misses        738      738
Flag Coverage Δ
integration 41.87% <ø> (ø)
unit 90.72% <ø> (ø)

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

Components Coverage Δ
NonTestCode 95.89% <ø> (ø)
OutsideTasks 95.72% <ø> (ø)
Files Coverage Δ
tasks/cache_test_rollups.py 94.87% <ø> (ø)

@github-actions GitHub Actions
Copy link

✅ All tests successful. No failed tests were found.

📣 Thoughts on this report? Let Codecov know! | Powered by Codecov

@joseph-sentry joseph-sentry added this pull request to the merge queue Nov 11, 2024
@codecov-qa
Copy link
codecov-qa bot commented Nov 11, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.91%. Comparing base (675b8ff) to head (256f06e).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #876   +/-   ##
=======================================
  Coverage   97.91%   97.91%           
=======================================
  Files         444      444           
  Lines       35399    35399           
=======================================
  Hits        34661    34661           
  Misses        738      738
Flag Coverage Δ
integration 41.87% <ø> (ø)
unit 90.72% <ø> (ø)

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

Components Coverage Δ
NonTestCode 95.89% <ø> (ø)
OutsideTasks 95.72% <ø> (ø)
Files with missing lines Coverage Δ
tasks/cache_test_rollups.py 94.87% <ø> (ø)

Merged via the queue into main with commit 5a2d60a Nov 11, 2024
26 of 27 checks passed
@joseph-sentry joseph-sentry deleted the joseph/fix-schema-error branch November 11, 2024 17:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@trent-codecov trent-codecov trent-codecov approved these changes

@ajay-sentry ajay-sentry Awaiting requested review from ajay-sentry

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@joseph-sentry @trent-codecov
0