Mahmoud
Mahmoud
I think the translated SQL query would be something like: ```python SELECT *, COUNT(skills) FROM Person GROUP BY skills HAVING COUNT(skills) > 1; ``` Or redis search version: ```python FT.AGGREGATE...
I think, at the moment, it is not possible to use this package in django because django migrations and redis-om migrator are two different things, that's because `HashModel` is not...
I think you forgot to run migrations first: ```python from fastapi import FastAPI from redis_om import ( Migrator ) app = FastAPI(title=__name__) @app.on_event("startup") async def startup(): await Migrator().run() ``` Update:...
Hey @johnson2427, just giving it a `default` value seems to solve the issue: ```python id: int = Field(index=True, default=0) ``` I dunno how it works. MAGIC. Update: I think there...
Y'all would laugh so hard if i should you the bug: https://github.com/redis/redis-om-python/blob/dcd84e0ab2686a3c6ff7f6c6a43ec49f4b65655f/aredis_om/model/model.py#L1210-L1213 It seems like the developer was trying to implement something, didn't like it, then forgot about removing the...
Time to run: ```sh git blame aredis_om/model/model.py -L 1210,1213 ``` ```sh 0990c2e1 redis_developer/orm/model.py (Andrew Brookins 2021-09-15 17:41:45 -0700 1210) try: d2fa4c58 redis_developer/model/model.py (Andrew Brookins 2021-10-20 13:01:46 -0700 1211) del fields["id"]...
fixed in [pull/337](https://github.com/redis/redis-om-python/pull/337).
I would like to work on this issue, however, i have a question related to the naming of the method, wanna call it `aggregate`? ~~Wanna support aggregations on both rows...
Hey @dvora-h. As you may know, the thing is that the `aggregate` function is a whole new query and not related/ doesn't require a `find` query. You can refer to...
> ensuring that we can do chained aggregations. I recently started to realize that. That's right! > The downside from approaches like this are reserved names, leading to Capitalized variables....