Improve kaggle system call

activeloopai / deeplake

Database for AI. Store Vectors, Images, Texts, Videos, etc. Use with LLMs/LangChain. Store, query, version, & visualize any AI data. Stream data in real-time to PyTorch/TensorFlow. https://activeloop.ai

https://activeloop.ai

Mozilla Public License 2.0

8.08k stars 616 forks source link

Improve kaggle system call #2892

Closed nvoxland-al closed 3 months ago

nvoxland-al commented 3 months ago

🚀 🚀 Pull Request

Impact

[X] Bug fix (non-breaking change which fixes expected existing functionality)
[ ] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Use subprocess.Popen instead of os.system

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is `33.33333%` with `4 lines` in your changes missing coverage. Please review.	Files with missing lines	Patch %	Lines
deeplake/auto/unstructured/kaggle.py	33.33%	4 Missing :warning:

:loudspeaker: Thoughts on this report? Let us know!

sonarcloud[bot] commented 3 months ago

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
33.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud