Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR consolidates and simplifies the indexing type system by unifying multiple specialized enum types (TextIndexEnumType, NumericIndexEnumType, EmbeddingIndexEnumType) into a single IndexType enum. It also adds support for new data types (Mesh, Audio, Video), enhances numeric type indexing capabilities, and includes various documentation improvements and bug fixes.
Key changes:
- Unified indexing system with a single
IndexTypeenum replacing multiple specialized enums - Added support for Mesh data type and expanded compression options
- Enhanced numeric types (Int8/16/32/64, UInt8/16/32/64, Float16/32/64) with optional indexing capabilities
- Improved documentation for index creation/dropping and various type functions
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| python/deeplake/types.pyi | Major refactoring of type system: unified IndexType enum, added new wrapper classes (TextIndex, NumericIndex, JsonIndex), enhanced numeric types with indexing support, expanded ClassLabel documentation, and added Mesh type |
| python/deeplake/types.py | Updated all exports to reflect new unified type system and removed deprecated enum types |
| python/deeplake/storage.pyi | Added async methods (get_async, set_async), new list_dirs method, constructors, and updated return types |
| python/deeplake/integrations/labelbox/labelbox_utils.py | Fixed grayscale image handling by adding explicit channel dimension normalization |
| python/deeplake/integrations/labelbox/converters.py | Changed polygon converter to use numpy arrays instead of nested lists |
| python/deeplake/init.pyi | Updated API signatures for create_index/drop_index with new unified types, enhanced documentation with more examples, added new exception types, new async methods, and additional repr methods |
| python/deeplake/init.py | Version bump to 4.4.1, added Windows platform check for fork handlers, and new exception exports |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
levonohanyan
approved these changes
Nov 21, 2025
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🚀 🚀 Pull Request
Impact
Description
Things to be aware of
Things to worry about
Additional Context