nikhil kandur
nikhil kandur
PR for issue #2685
yeah, this PR only changed the typo error of `start` to `end` and nothing with the logic of first week calculation. Looking into @nawans comment, I said it here, [2685#issuecomment-892932735](https://github.com/microsoft/Recognizers-Text/issues/2685#issuecomment-892932735)...
And also, I think, we need to revisit quarters calculation as well, as the python one may not be following ISO standards.
This line [datatypes_timex_expression/timex_resolver.py#L167](https://github.com/microsoft/Recognizers-Text/blob/master/Python/libraries/datatypes-timex-expression/datatypes_timex_expression/timex_resolver.py#L167) should be of `end` datetime rather than `start` datetime i.e., ```python Timex(year=end.year, month=end.month, day_of_month=end.day) ```
@nawanas - I have my **python** code snippet above and I have tried using `datatypes-timex-expression==1.0.2a2` library and pointed out the error line as well. I'm able to consistently reproduce the...
It errors out for the extreme end of the year/month calculation of a week. I tried for 'this week' and end date is incorrect as well. 
All other weekdays work except the checking day is same as that weekday. I'm currently reporting only if you're checking for the weekday same as the day you're checking. Here...
When I check in Python using datatypes-timex-expression==1.0.2.a2 it is similar to what I mentioned in the issue description, ```python from datatypes_timex_expression import TimexResolver from datetime import datetime today = datetime.today()...
Interestingly, I'm also getting the same error. I have `rq` for async workflow which also installs redis. Even with the redisearch installation, i'm still getting the same error. ```bash $...
> @nikhilkandur Please do `pip uninstall redisearch` and do `pip install redis==4.5.5` that'll solve the issue. I installed `redis==4.5.5` but i'm getting below error but that version of [redisearch](https://pypi.org/project/redisearch/#history) doesn't...