Skip to content

feat: file object data type - #291

Merged
cofin merged 59 commits into
mainfrom
file-type
Apr 6, 2025
Merged

cofin merged 59 commits into
mainfrom
file-type

Conversation

@cofin

@cofin cofin commented Nov 15, 2024 •

Copy link
Copy Markdown
Member

Description

Implement a file data type that leverages obstore or fsspec. Supports any supported FSSpec or Obstore backend it including sftp, gcs, s3, local, and more.

Closes

@cofin cofin linked an issue Nov 15, 2024 that may be closed by this pull request
Comment thread advanced_alchemy/types/file_object.py Outdated
Comment thread advanced_alchemy/types/file_object.py Outdated
Comment thread advanced_alchemy/types/file_object/backends/fsspec.py Outdated
Comment thread advanced_alchemy/types/file_object/backends/fsspec.py Outdated
Comment thread advanced_alchemy/types/file_object/backends/fsspec.py Outdated
@codecov-commenter

codecov-commenter commented Apr 2, 2025 •

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 80.74454% with 150 lines in your changes missing coverage. Please review.

Project coverage is 87.81%. Comparing base (dfc0f7f) to head (14681eb).

Files with missing lines Patch % Lines
...anced_alchemy/types/file_object/session_tracker.py 53.94% 30 Missing and 5 partials ⚠️
advanced_alchemy/_listeners.py 77.04% 19 Missing and 9 partials ⚠️
...nced_alchemy/types/file_object/backends/obstore.py 72.27% 18 Missing and 10 partials ⚠️
...anced_alchemy/types/file_object/backends/fsspec.py 79.64% 11 Missing and 12 partials ⚠️
advanced_alchemy/types/file_object/data_type.py 70.17% 12 Missing and 5 partials ⚠️
advanced_alchemy/types/file_object/_utils.py 66.66% 7 Missing and 3 partials ⚠️
advanced_alchemy/types/file_object/file.py 95.97% 3 Missing and 3 partials ⚠️
advanced_alchemy/types/file_object/base.py 92.68% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #291      +/-   ##
==========================================
- Coverage   89.03%   87.81%   -1.22%     
==========================================
  Files          68       78      +10     
  Lines        4358     5106     +748     
  Branches      529      636     +107     
==========================================
+ Hits         3880     4484     +604     
- Misses        328      426      +98     
- Partials      150      196      +46     

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

cofin and others added 3 commits April 4, 2025 20:12
Signed-off-by: Cody Fincher <204685+cofin@users.noreply.github.com>
@cofin
cofin requested a review from provinzkraut April 5, 2025 03:15
@sonarqubecloud

sonarqubecloud Bot commented Apr 6, 2025

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Apr 6, 2025

Copy link
Copy Markdown

Documentation preview will be available shortly at https://litestar-org.github.io/advanced-alchemy-docs-preview/291

@cofin
cofin merged commit ef95fe8 into main Apr 6, 2025
@cofin
cofin deleted the file-type branch April 6, 2025 22:05
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.

Enhancement: Create a file field type for SQLAlchemy

3 participants