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
[small] remove chunk sizes from htypes #1037
Conversation
… into task/uncompressed-is-None
Codecov Report
@@ Coverage Diff @@
## main #1037 +/- ##
==========================================
+ Coverage 88.61% 88.67% +0.06%
==========================================
Files 87 86 -1
Lines 3953 3939 -14
==========================================
- Hits 3503 3493 -10
+ Misses 450 446 -4
Continue to review full report at Codecov.
|
hub/htypes.py
Outdated
@@ -42,7 +41,6 @@ | |||
|
|||
# these configs are added to every `htype` | |||
COMMON_CONFIGS = { | |||
"chunk_size": DEFAULT_MAX_CHUNK_SIZE, | |||
"chunk_compression": DEFAULT_CHUNK_COMPRESSION, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
chunk compression is removed in #1038
update sample compression API
hub/util/exceptions.py
Outdated
@@ -309,12 +309,23 @@ def __init__(self, htype: str, available_htypes: Sequence[str]): | |||
class TensorMetaInvalidHtypeOverwriteValue(MetaError): | |||
def __init__(self, key: str, value: Any, explanation: str = ""): | |||
super().__init__( | |||
"Invalid value {} for tensor meta key {}. {}".format( | |||
"Invalid value '{}' for tensor meta key {}. {}".format( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's stick to fstrings everywhere so we don't need to do fstring replacement PRs in the future.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
was an upstream change i just pulled in
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
also removed
hub.core.typing
since it wasn't really being used