Ilya Priven

Results 164 comments of Ilya Priven

As I expected, there was implied use of `to_json()`. You never mentioned it up until now. For now, `attributes_to_get` doesn't actually manifest in any way on the model. I'd suggest...

Initially I'd recommend: ```python models = TestersModel.query(hash_key="class_id") return { "statusCode": status_code, "body": [ { "tester_id": model.tester_id, "tester_name": model.tester_name, } for model in models ], } ``` Only then ask yourself:...

What I'm trying to suggest is that you might be complicating your code for what's not a meaningful amount of traffic. As for read capacity, anything under 4K is still...

There are many built-in immutable types. We can add this to `Attribute.__init__`: ```python if not callable(default) and not isinstance(default, IMMUTABLE_TYPES): raise ValueError(...) ``` This would address the common "mutable dict/list/set"...

In each case it seems that the data has been populated in DynamoDB through some other means than PynamoDB's `UTCTimeDateAttribute`. We probably don't want to make a flexible parser to...

Curious why you decided to stop maintaining this. Was it a one-time experiment? Or the rate of changes and proprietary nature of IntelliJ make the effort not gratifying?

I haven't tested it out, but looking at the source, will overriding `BooleanAttribute`'s `get_value` work? ```py class CustomAttribute(BooleanAttribute): def get_value(self, value: Dict[str, Any]) -> Any: if 'N' in value: value...

Slightly shorter repro: ```python from __future__ import annotations from typing import overload, Generic, TypeVar T = TypeVar('T') class C(Generic[T]): @overload # type:ignore[misc] @property def f(self: C[int]) -> int: raise NotImplementedError...

Probably can be done by making these lines not run and have `converted = None` instead: https://github.com/python/mypy/blob/70d4bb27a2733e43958dc5046befdc16ed297a5c/mypy/fastparse.py#L364-L366

Another category of bugs this could help is arithmetics between naive and tz-aware `datetime`s.