Skip to content
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

feat: 🎸 10x the size of supported images #1392

Merged
merged 1 commit into from
Jun 19, 2023

Conversation

severo
Copy link
Collaborator

@severo severo commented Jun 19, 2023

No description provided.

@codecov-commenter
Copy link

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (170b1c6) 91.58% compared to head (7b9d1a7) 91.58%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1392   +/-   ##
=======================================
  Coverage   91.58%   91.58%           
=======================================
  Files          70       70           
  Lines        4422     4422           
=======================================
  Hits         4050     4050           
  Misses        372      372           
Flag Coverage Δ
services_worker 91.58% <100.00%> (ø)

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

Impacted Files Coverage Δ
services/worker/src/worker/utils.py 87.50% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@severo severo merged commit b7f13c8 into main Jun 19, 2023
@severo severo deleted the increase-size-of-supported-images branch June 19, 2023 12:36
AndreaFrancis added a commit that referenced this pull request Jun 27, 2023
… DuckDB (#1296)

* Draft files

* Adding duckdb index job runner

* Fix style

* WIP adding fts on API

* Remove non used code

* Fix style

* Adding chart objects

* Rollback dependency in API

* Depend on parquet an split

* Fix libcommon test

* Send index file to dedicated branch

* Fix test in first parquet

* Fix merge hanges

* Fix poetry files

* Adding happy path test

* Adding other test scenarios

* Adding chart configuration

* Apply suggestions from code review

Co-authored-by: Sylvain Lesage <sylvain.lesage@huggingface.co>

* Change ParquetFileItem to SplitHubFile

* Inherit from SplitCachedJobRunner

* Fix style

* Depends on info  featues instead of parquet schema

* Fix libcommon test

* Apply code review suggestions

* Some details

* Fix style

* Fix test

* Apply code review suggestions

* Update chart/values.yaml

Co-authored-by: Sylvain Lesage <sylvain.lesage@huggingface.co>

* Apply suggestions from code review

Co-authored-by: Sylvain Lesage <sylvain.lesage@huggingface.co>

* Apply code review suggestions

* [docs] Improvements (#1376)

* add end-to-end example

* apply feedback

* Fix closing brackets and GH action link (#1389)

* Fix typo in erro rmessage (#1391)

* Add docker internal to extra_hosts (#1390)

* fix: 🐛 support bigger images (#1387)

* fix: 🐛 support bigger images

fixes #1361

* style: 💄 fix style

* style: 💄 add types for Pillow

* Rename dev to staging, and use staging mongodb cluster (#1383)

* chore: 🤖 remove makefile targets

since we use ArgoCD now

* feat: 🎸 align dev on prod, and use secret for mongo url

* feat: 🎸 rename dev to staging

* ci: 🎡 change dev to staging in ci

* feat: 🎸 10x the size of supported images (#1392)

* Fix exception

* Fix test in libcommon

* Apply some code review suggestions

* Apply code review suggestions

* Adding close connection

* Upgrade duckdb version

* Apply code review suggestions

* Fix style

* Adding some test cases

* Remove duplicate code by merge

* Fix imports

* Apply code review suggestions

* Apply suggestions from code review

Co-authored-by: Sylvain Lesage <sylvain.lesage@huggingface.co>

* Add test

---------

Co-authored-by: Sylvain Lesage <sylvain.lesage@huggingface.co>
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
Co-authored-by: Bas Krahmer <baskrahmer@gmail.com>
Co-authored-by: Albert Villanova del Moral <8515462+albertvillanova@users.noreply.github.com>
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